Skip to content

Conversation

@v-i-s-h-n-u-ps
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings October 15, 2025 07:20
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adds new quiz response fields to type definition tests and updates quiz question tests to reflect API changes, including handling of skippable questions and metadata, plus adds a rejection test for quizzes with jump logic.

  • Added is_skippable to NextQuestionResponse test fixtures and metadata to QuizResultsConfigResponse.
  • Updated expected total_questions count and added a new rejection test for jump-logic quizzes.
  • Introduced an inadvertent focus (describe.only) limiting test execution scope.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/types/tests/quizzes.test-d.ts Added new properties (is_skippable, metadata) to strengthen type coverage.
spec/src/modules/quizzes.js Modified quiz tests: focused suite with describe.only, adjusted expectations, added new error test.

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@esezen esezen merged commit 00306d2 into Constructor-io:master Oct 15, 2025
6 of 9 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.

2 participants