-
-
Notifications
You must be signed in to change notification settings - Fork 338
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 scenario testing framework, initial test #4025
Conversation
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.
@mathemancer I've added these tests to the CI, however there are some tests that are failing and also I see some stale print statement in tests. Can you please fix the tests and remove those print statements?
Update: @mathemancer The tests seem to be failing because of our new Users APIs, I'll fix the tests. |
Ah, I forgot about that! Well, the tests are catching what they should, at least. |
Related to #4017
This adds tests for the code in #4021, and sets up a framework for scenario testing.
Run the tests with the new
run_api_tests.sh
script in a root window, one that has docker permissions, or usingsudo
.Technical details
I'm going to leave this as a draft so @Anish9901 and I can discuss. If we decide to proceed, we should add this into the CI testing matrix pipeline.
Screenshots
Checklist
Update index.md
).develop
branch of the repositoryvisible errors.
Developer Certificate of Origin
Developer Certificate of Origin