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(Multiple Choice): Add tests to Multiple Choice authoring #1435

Merged
merged 4 commits into from
Oct 13, 2023

Conversation

geoffreykwan
Copy link
Member

Changes

Added tests to Multiple Choice authoring.

Test

Only test files were changed/added so functionality should not have changed.

@geoffreykwan geoffreykwan self-assigned this Sep 26, 2023
@geoffreykwan geoffreykwan marked this pull request as ready for review September 26, 2023 19:44
@geoffreykwan geoffreykwan requested a review from breity September 26, 2023 19:45
Copy link
Member

@breity breity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test are working. 👍

A couple thoughts:

  • Could maybe reorganize to better follow the Arrange-Act-Assert pattern by moving the actions out of the it blocks and into the beforeEach blocks, leaving only assertions in the it blocks.
  • Wondering if we need all the click action function wrappers that are only used once. Could you call the click actions directly?

These are probably issues we should discuss, agree on conventions for test structures. So maybe okay to move forward here either way.

@geoffreykwan geoffreykwan force-pushed the add-multiple-choice-authoring-tests branch from ae98068 to 88de34e Compare October 13, 2023 16:57
@geoffreykwan geoffreykwan merged commit 517c925 into develop Oct 13, 2023
2 checks passed
@geoffreykwan geoffreykwan deleted the add-multiple-choice-authoring-tests branch October 13, 2023 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants