Skip to content

[MBL-19758][Teacher] Unstub SpeedGrader related interaction tests#3520

Open
adamNagy56 wants to merge 6 commits intomasterfrom
MBL-19758-Unstub-Speedgrader-related-interaction-tests
Open

[MBL-19758][Teacher] Unstub SpeedGrader related interaction tests#3520
adamNagy56 wants to merge 6 commits intomasterfrom
MBL-19758-Unstub-Speedgrader-related-interaction-tests

Conversation

@adamNagy56
Copy link
Contributor

Summary

Refactor and unstub SpeedGrader interaction tests through the Teacher app.

refs: MBL-19758
affects: Teacher
release note

@github-actions
Copy link

github-actions bot commented Feb 10, 2026

📊 Code Coverage Report

✅ Student

  • PR Coverage: 43.16%
  • Master Coverage: 43.16%
  • Delta: +0.00%

✅ Teacher

  • PR Coverage: 25.39%
  • Master Coverage: 25.39%
  • Delta: +0.00%

✅ Pandautils

  • PR Coverage: 23.20%
  • Master Coverage: 23.20%
  • Delta: +0.00%

📈 Overall Average

  • PR Coverage: 30.58%
  • Master Coverage: 30.58%
  • Delta: +0.00%

@github-actions
Copy link

github-actions bot commented Feb 10, 2026

🧪 Unit Test Results

✅ 📱 Teacher App

  • Tests: 373 total, 0 failed, 0 skipped
  • Duration: 33.312s
  • Success Rate: 100%

✅ 📦 Submodules

  • Tests: 2983 total, 0 failed, 0 skipped
  • Duration: 48.046s
  • Success Rate: 100%

📊 Summary

  • Total Tests: 3356
  • Failed: 0
  • Skipped: 0
  • Status: ✅ All tests passed!

Last updated: Thu, 12 Feb 2026 14:11:13 GMT

@github-actions
Copy link

github-actions bot commented Feb 10, 2026

Teacher Install Page

refs: MBL-19758
affects: Teacher
release note:
…ments and media recordings.

refs: MBL-19758
affects: Teacher
release note:
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.

Basically LGTM! Please do the goToSpeedGraderGradePage refactor to give back MockCanvas object (and you can do the rest, tiny stuff as well then).

refs: MBL-19758
affects: Teacher
release note:
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