Closed suryabulusu closed 8 months ago
Attention: 32 lines
in your changes are missing coverage. Please review.
Comparison is base (
2c8062b
) 78.75% compared to head (3554e86
) 79.40%.
Files | Patch % | Lines |
---|---|---|
src/components/Omr/OmrItem.vue | 68.33% | 19 Missing :warning: |
src/components/Questions/Body.vue | 70.45% | 13 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Fixes #119
Related notion note: https://www.notion.so/avantifellows/JEE-Advanced-New-Question-Types-a315a0fdace547069bccd41d13fe3f62 (see for screenshots and staging test links)
Summary
Adds a new question type for advanced matrix matching in assessment and omr-assessment modes. I explain major updates and files changed below:
types.ts
: acceptstring[]
as a possible answer type, since matrix matching answers are of the form["AP", "BR", "CT"]
, etc.Utilities.ts
: add answer eval formatrix-match
type; very similar tomulti-choice
.Body.vue
andOmrItem.vue
: howmatrix-match
type's options look like; table structure, option selection, updates to parent component, CSSQuestionModal.vue
andOmrModal.vue
: how session answers get updated formatrix-match
questions when they receive emits from child componentsassessment_quiz.json
andmulti_qset_omr.json
: updates to fixtures for cypress testing ofmatrix-match
typerenders-player-assessment.js
andrenders-omr-assessment.js
: updates to spec files for cypress testing ofmatrix-match
typeTest Plan
docs/ENV.md
,.env.example
and the Github workflows.~alt
attributes~<img>
tags havewidth
andheight
specified~target="_blank"
links haverel="noopener"
~aria-label
attributes set~