Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[SDS-794] Check for raw data list was wrong #150

Merged
merged 1 commit into from
Oct 24, 2022
Merged

[SDS-794] Check for raw data list was wrong #150

merged 1 commit into from
Oct 24, 2022

Conversation

QFer
Copy link
Contributor

@QFer QFer commented Oct 24, 2022 •
edited
Loading

  • Assumed 'empty' raw data list was wrong. The multi measurement empty raw_data_list is [[]] instead of []
  • Adjusted test cases
  • Bug fixed. First check to see if there is a list of lists and then if the first list is filled with data

@coveralls
Copy link
Collaborator

Coverage Status

Coverage remained the same at 100.0% when pulling 79941c9 on fix/sds-794/failing_integration_tests into 9e401a9 on dev.

1 similar comment
@coveralls
Copy link
Collaborator

Coverage Status

Coverage remained the same at 100.0% when pulling 79941c9 on fix/sds-794/failing_integration_tests into 9e401a9 on dev.

@QFer QFer requested a review from tomrijnbeek October 24, 2022 09:54
@QFer QFer merged commit 3054953 into dev Oct 24, 2022
@QFer QFer deleted the fix/sds-794/failing_integration_tests branch October 24, 2022 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tomrijnbeek tomrijnbeek tomrijnbeek approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@QFer @coveralls @tomrijnbeek