Skip to content

Conversation

adamNagy56
Copy link
Contributor

Checklist

  • Run E2E test suite

refs: MBL-16336
affects: Student
release note:

@adamNagy56 adamNagy56 requested a review from Copilot August 13, 2025 11:24
Copy link

@Copilot 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

This PR enhances the classic quizzes E2E test to include search functionality testing and removes the @Stub annotation to enable the test. The changes focus on maintaining and improving test coverage for the quiz list search capabilities in the Student app.

  • Add search functionality testing to the QuizListPage
  • Update QuizzesE2ETest to test search behavior for published and unpublished quizzes
  • Remove @Stub annotation and improve test stability with better timing controls

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
StudentTest.kt Updates QuizListPage initialization to include search configuration
QuizListPage.kt Adds search functionality methods and search-related UI element handling
QuizzesE2ETest.kt Removes stub annotation, adds search testing, and improves test flow with better timing

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

refs: MBL-16336
affects: Student
release note:
@inst-danger
Copy link
Contributor

Student Install Page

@inst-danger
Copy link
Contributor

Student Install Page

Copy link
Contributor

@kdeakinstructure kdeakinstructure left a comment

Choose a reason for hiding this comment

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

Added some comments.

@inst-danger
Copy link
Contributor

Student Install Page

@inst-danger
Copy link
Contributor

Student Install Page

@inst-danger
Copy link
Contributor

Student Install Page

@inst-danger
Copy link
Contributor

Student Install Page

@adamNagy56 adamNagy56 merged commit 758efeb into master Aug 27, 2025
1 check passed
@adamNagy56 adamNagy56 deleted the Maintain-ClassicQuizzes-E2Etest branch August 27, 2025 13:06
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.

4 participants