-
Notifications
You must be signed in to change notification settings - Fork 4
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
refactor: update integration tests workflow and add pytest configuration #221
Conversation
daac8ac
to
292c269
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.
Thanks @magajh, LGTM! Just remember to update the branch of the integration tests action.
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.
My only comment here is we can use the main or release version of the integration test action. The rest looks good to me. Thanks ✨
@MaferMazu @BryanttV thank you both for the review! I've updated the action branch to main and removed the input |
… version for integration tests
add pylint constraint
f1bc45f
to
d9325c8
Compare
Description
This PR includes some necessary changes to use the updated integration testing action https://github.com/eduNEXT/integration-test-in-tutor. The changes are described below:
pytest
andpytest-django
requirements to run the tests.pytest
for running unit tests, ensuring consistency in how tests are executed across all plugins.integration
folders.For more information about the changes and the reasoning behind them, please refer to the documentation on writing integration tests for the plugins: https://internal.docs.edunext.co/en/latest/internal/Teams/Descontrol/Internal_processes/integration_tests_in_plugins/integration_tests_with_openedx_components.html
Note for Reviewers
This PR does not move the unit test files to
unit/
folders. That effort will be part of a future PR when integration tests are added to the plugin.How to Test
Ensure that both unit and integration tests pass without issues in the GitHub CI/CD pipeline.