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

✅(tests) add open response assessment tests #25

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

Conversation

SergioSim
Copy link
Collaborator

@SergioSim SergioSim commented Nov 8, 2022

Purpose

We want to generate some of the available edX open response assessment events.

Proposal

Generate the following events:

  • openassessmentblock.save_submission
  • openassessmentblock.create_submission
  • openassessmentblock.get_peer_submission
  • openassessmentblock.get_submission_for_staff_grading
  • openassessmentblock.self_assess
  • openassessmentblock.staff_assess
  • openassessmentblock.submit_feedback_on_assessments
  • openassessment.upload_file
  • openassessmentblock.save_files_descriptions
  • openassessment.student_training_assess_example
  • openassessmentblock.peer_assess

Includes changes from #22, #23 and #24. Blocked until #22, #23 and #24 are merged.

When bootstraping this project in the dev branch two additional
directories are created: edx/src and edx/modules.
To ease switching between a feature branch and the dev branch we
choose to ignore the edx/modules directory.
When bootstraping this project in the dev branch two additional
directories are created: edx/src and edx/modules.
To ease switching between a feature branch and the dev branch we
choose to make the linter ignore the edx directory.
To keep the circleci machine up to date.
@SergioSim SergioSim self-assigned this Nov 8, 2022
@SergioSim SergioSim force-pushed the ora2 branch 6 times, most recently from 1ecd96e to baaab88 Compare November 10, 2022 14:09
The iframe in the js_input_response test needs some time to become
interactive. Therefore we disable chromeWebSecurity to give cypress
access to the cross-domain iframe to check whether the iframe is ready
or not.
The drag_and_drop test relies on the problem beeing at it's initial
state.
However, edX preserves the state of the problem once it's submitted.
Therefore we reset the drag_and_drop problem before answering it.
We upgrade cypress to its latest version.
We also upgrade the javascript dependencies used for linting.
We intend in the future to generate open response assessments related
edX events.
Therefore we need to be able to create multiple edX users during test
executions to simulate peer-graded assessments.
Also, using generated users in our tests makes them more isolated.
@SergioSim SergioSim force-pushed the ora2 branch 2 times, most recently from 93af236 to 20eefc9 Compare November 16, 2022 09:29
@SergioSim SergioSim force-pushed the ora2 branch 4 times, most recently from 0685e01 to 394f1af Compare December 7, 2022 11:21
@SergioSim SergioSim force-pushed the ora2 branch 2 times, most recently from 8006e08 to edcaa3c Compare December 8, 2022 11:50
We want to generate some of the available edX open response assessment
events.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant