-
Notifications
You must be signed in to change notification settings - Fork 9
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
test: add integration tests for the grade api AP-1406
#302
Conversation
AP-1406
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.
Thank you, @jignaciopm! I've added some additional comments for you to review. Please let me know once you do :)
@jignaciopm integration tests are failing. Have you checked why we are getting this error? https://github.com/eduNEXT/eox-core/actions/runs/11840999793/job/32996180959?pr=302#step:2:5674 Also, we need to rebase with master to get the latest changes here |
c300b4b
to
b2086c7
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.
Thank you for addressing all of our comments!
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.
@jignaciopm I'd like to merge this PR first #308, so we can make sure all the integration tests here are passing except for the 'nightly' version
df9026a
to
ca0f710
Compare
… in the documentation
ca0f710
to
0f5bfcb
Compare
Description
This PR adds integration tests for the Grade API and remove old module test integrations. These tests are executed in the job of Tutor Integration Tests
Testing instructions
Check the jobs of Tutor Integration Tests in the PR.
Jira Issue
Checklist for Merge