[MBL-19970][All] - Fix Help Menu E2E tests#3664
Merged
kdeakinstructure merged 4 commits intomasterfrom Apr 29, 2026
Merged
Conversation
…s and Intent URLS) refs: MBL-19970 affects: Parent, Student, Teacher release note: -
Contributor
🧪 Unit Test Results✅ 📱 Parent App
✅ 📱 Student App
✅ 📱 Teacher App
✅ 🌅 Horizon
✅ 📦 Submodules
📊 Summary
Last updated: Wed, 29 Apr 2026 07:48:25 GMT |
Contributor
📊 Code Coverage Report✅ Student
✅ Teacher
✅ Pandautils
📈 Overall Average
|
Contributor
|
Parent Install Page |
Contributor
|
Student Install Page |
Contributor
|
Teacher Install Page |
There was a problem hiding this comment.
Pull request overview
Updates Help Menu E2E test selectors and expectations to match recent Help Menu copy/link changes across Parent, Student, and Teacher apps.
Changes:
- Replaced the “Submit a Feature Idea” Help Menu entry strings with “Share a Contribution” in shared test constants.
- Updated Help Page test page objects to locate/assert the renamed Help Menu item.
- Updated Help Menu E2E tests to expect the new “Share a Contribution” destination URL.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| automation/espresso/src/main/kotlin/com/instructure/canvas/espresso/StringConstants.kt | Renames shared Help Menu string constants used by UI/E2E tests. |
| apps/teacher/src/androidTest/java/com/instructure/teacher/ui/pages/classic/HelpPage.kt | Updates Teacher Help dialog selectors/assertions to the new menu item title/subtitle. |
| apps/teacher/src/androidTest/java/com/instructure/teacher/ui/e2e/classic/HelpMenuE2ETest.kt | Updates expected intent URL for the renamed menu item. |
| apps/student/src/androidTest/java/com/instructure/student/ui/pages/classic/HelpPage.kt | Updates Student Help dialog selectors/assertions to the new menu item title/subtitle. |
| apps/student/src/androidTest/java/com/instructure/student/ui/e2e/classic/HelpMenuE2ETest.kt | Updates expected intent URL for the renamed menu item. |
| apps/parent/src/androidTest/java/com/instructure/parentapp/ui/pages/classic/HelpPage.kt | Updates Parent Help dialog selectors/assertions to the new menu item title/subtitle. |
| apps/parent/src/androidTest/java/com/instructure/parentapp/ui/e2e/classic/HelpMenuE2ETest.kt | Updates expected intent URL for the renamed menu item. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
adamNagy56
approved these changes
Apr 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix breaking Help Menu E2E tests in all the apps (Changed some strings and Intent URLS)
refs: MBL-19970
affects: Parent, Student, Teacher
release note: -