-
Notifications
You must be signed in to change notification settings - Fork 5
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.
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
Add system tests #611
Add system tests #611
Conversation
…e bluesky stomp library
Closes #597 --------- Co-authored-by: Callum Forrester <callum.forrester@diamond.ac.uk>
9e70c3e
to
aef0f7d
Compare
aef0f7d
to
bcc8e57
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #611 +/- ##
==========================================
- Coverage 92.52% 92.52% -0.01%
==========================================
Files 35 35
Lines 1633 1632 -1
==========================================
- Hits 1511 1510 -1
Misses 122 122 ☔ View full report in Codecov by Sentry. |
I have not move the current test to unit test will move after #589 is merge ... |
b9bb6dd
to
15a5c93
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good start, @ZohebShaikh
created_task = rest.create_task(task) | ||
created_tasks.append(created_task) | ||
|
||
task_list = get_response(rest._url("/tasks"), TasksListResponse) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm unsure why the client object doesn't provide what you want here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have added code for this here b5bb825
I think we should change
get_task(task_id) to get_task_by_id(task_id) -> assert task_id is not ""
and get_all_tasks()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was added by this #508
Replaced by #618 for neatness |
No description provided.