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

assert_all_tasks_processed should be moved to a separate module #362

Closed
mateuszsrebrny opened this issue May 5, 2021 · 0 comments · Fixed by #516
Closed

assert_all_tasks_processed should be moved to a separate module #362

mateuszsrebrny opened this issue May 5, 2021 · 0 comments · Fixed by #516
Assignees
@azawlocki

Comments

@mateuszsrebrny
Copy link
Contributor

The same function is defined in test_run_blender.py. We should make ALL_TASKS a parameter and move the function to a separate module so it could be imported in both test_run_blender.py and test_run_yacat.py.

The same goes for assert_no_errors and assert_all_invoices_accepted.

Originally posted by @azawlocki in #344 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@azawlocki azawlocki

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Integration test for a Simple Service example golemfactory/yapapi
2 participants
@azawlocki @mateuszsrebrny