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.

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

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

jignaciopm
Copy link
Contributor

@jignaciopm jignaciopm commented Nov 7, 2024

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

  • Tested in a remote environment
  • Updated documentation
  • Rebased master/main
  • Squashed commits

@jignaciopm jignaciopm requested a review from a team as a code owner November 7, 2024 03:26
@jignaciopm jignaciopm requested a review from magajh November 7, 2024 03:27
@jignaciopm jignaciopm changed the title Jipm/ap 1406 test: add integration tests for the grade api AP-1406 Nov 7, 2024
eox_core/api/v1/tests/integration/test_views.py Outdated Show resolved Hide resolved
eox_core/api/v1/tests/integration/test_views.py Outdated Show resolved Hide resolved
eox_core/api/v1/tests/integration/test_views.py Outdated Show resolved Hide resolved
eox_core/api/v1/tests/integration/test_views.py Outdated Show resolved Hide resolved
eox_core/api/v1/tests/integration/test_views.py Outdated Show resolved Hide resolved
eox_core/api/v1/tests/integration/test_views.py Outdated Show resolved Hide resolved
eox_core/api/v1/tests/integration/test_views.py Outdated Show resolved Hide resolved
eox_core/tests/integration/README.rst Show resolved Hide resolved
Copy link
Contributor

@mariajgrimaldi mariajgrimaldi left a 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 :)

eox_core/api/v1/tests/integration/data/fake_users.py Outdated Show resolved Hide resolved
eox_core/api/v1/tests/integration/test_views.py Outdated Show resolved Hide resolved
@magajh
Copy link
Contributor

magajh commented Nov 20, 2024

@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

.github/workflows/integration-test.yml Outdated Show resolved Hide resolved
@jignaciopm jignaciopm requested a review from magajh November 25, 2024 18:15
@jignaciopm
Copy link
Contributor Author

Copy link
Contributor

@mariajgrimaldi mariajgrimaldi left a 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!

Copy link
Contributor

@magajh magajh left a 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

@jignaciopm jignaciopm force-pushed the jipm/AP-1406 branch 4 times, most recently from df9026a to ca0f710 Compare November 27, 2024 21:25
@jignaciopm jignaciopm merged commit 9aef946 into master Nov 27, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants