Skip to content

Add timeseries CDA testing - #179

Merged
tlelv merged 20 commits into
mainfrom
timeseries_cda_testing
Aug 27, 2025
Merged

Add timeseries CDA testing#179
tlelv merged 20 commits into
mainfrom
timeseries_cda_testing

Conversation

@tlelv

@tlelv tlelv commented Aug 22, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

@Enovotny Enovotny left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove the store timeseries from the setup_data.

order the test this way

  1. test_store_timeseries. This will add the single timeseries that can be used in the following tests.
  2. test_get_timeseries. This call is missing and should test getting the above timeseries.
    3.) test_timeseries_df_to_json.
  3. test_store_multi_timeseries. Add a second timeseries to the call so it tests adding multiple timeseries. you can just change the version of the tsid to something different. Like Rev-Test or something like that
  4. test_get_multi_timesereis. Call the timeseries you stored in the previous store_mulit_Call.
  5. test_delete_timeseries. Delete one of the timeseries that have been stored.

@tlelv
tlelv requested a review from Enovotny August 22, 2025 19:29
@sonarqubecloud

Copy link
Copy Markdown

@tlelv
tlelv merged commit f79f44f into main Aug 27, 2025
9 checks passed
@tlelv
tlelv deleted the timeseries_cda_testing branch August 27, 2025 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants