Skip to content

Conversation

szabinst
Copy link
Collaborator

@szabinst szabinst commented Oct 6, 2025

  • Adds the capability to create snapshot tests
  • Adds a few tests to cover the Account tab in Canvas Career.

[ignore-commit-lint]
builds: Student

@inst-danger
Copy link
Contributor

inst-danger commented Oct 6, 2025

Fails
🚫 Build failed, skipping coverage check
❌ XCTest failed: HorizonSnapshotTests/AccountViewSnapshotTests/testAccountViewAccessibility
Attached Failure Diff
Attached Failure Diff
Attached Failure Diff
❌ XCTest failed: HorizonSnapshotTests/AccountViewSnapshotTests/testAccountViewDefault
Attached Failure Diff
Attached Failure Diff
❌ XCTest failed: HorizonSnapshotTests/AccountViewSnapshotTests/testAccountViewLoading
Attached Failure Diff
Attached Failure Diff
❌ XCTest failed: HorizonSnapshotTests/AccountViewSnapshotTests/testAccountViewNoExperienceSwitch
Attached Failure Diff
Attached Failure Diff
❌ XCTest failed: HorizonSnapshotTests/NotificationSettingsViewSnapshotTests/testNotificationSettingsViewDefault
Attached Failure Diff
Attached Failure Diff
❌ XCTest failed: HorizonSnapshotTests/NotificationSettingsViewSnapshotTests/testNotificationSettingsViewLoading
Attached Failure Diff
Attached Failure Diff
❌ XCTest failed: HorizonSnapshotTests/NotificationSettingsViewSnapshotTests/testNotificationSettingsViewNoPushConfigured
Attached Failure Diff
Attached Failure Diff
❌ XCTest failed: HorizonSnapshotTests/NotificationSettingsViewSnapshotTests/testNotificationSettingsViewPushDisabled
Attached Failure Diff
Attached Failure Diff
❌ XCTest failed: HorizonSnapshotTests/ProfileAdvancedViewSnapshotTests/testProfileAdvancedViewDefault
Attached Failure Diff
Attached Failure Diff
❌ XCTest failed: HorizonSnapshotTests/ProfileAdvancedViewSnapshotTests/testProfileAdvancedViewLoading
Attached Failure Diff
Attached Failure Diff
❌ (and 3 other test failures)

Generated by 🚫 dangerJS against c6f3fb5

@inst-danger
Copy link
Contributor

Builds

Commit: fix: Student project-ci.yml (c6f3fb5)
Build Number: 493
Built At: Oct 06 17:37 CEST (10/06 09:37 AM MDT)

Student

@szabinst szabinst marked this pull request as draft October 6, 2025 16:21
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