Skip to content

WS-2782: Fix failing darkUI checks - #14386

Merged
alex-magana merged 3 commits into
WS-2782-apply-dark-ui-for-topic-pagesfrom
WS-2782-update-tests
Sep 11, 2026
Merged

WS-2782: Fix failing darkUI checks #14386
alex-magana merged 3 commits into
WS-2782-apply-dark-ui-for-topic-pagesfrom
WS-2782-update-tests

Conversation

@alex-magana

@alex-magana alex-magana commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Resolves JIRA: WS-2782

Summary

Resolve and update failing PR checks in #14372

Code changes

  • Update snapshots to reflect styling changes made on the video media icon.
  • Update src/app/contexts/RequestContext/index.tsx to assert passing of primaryMediaType.
Screenshot 2026-09-11 at 10 50 46

Testing

  1. N/A

Useful Links

Copilot AI lite review requested due to automatic review settings September 11, 2026 06:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

Snapshot-only updates address the dark UI check failures with no runtime changes.

Pull request overview

Refreshes canonical Jest snapshots to resolve dark UI check failures caused by updated Emotion-generated media icon classes.

Changes:

  • Updates five integration snapshots.
  • No runtime code changes.
File summaries
File Description
ws-nextjs-app/integration/pages/onDemandTVPage/pashtoBrand/__snapshots__/canonical.test.ts.snap Updates Pashto TV snapshot classes
ws-nextjs-app/integration/pages/onDemandTVPage/hausa/__snapshots__/canonical.test.ts.snap Updates Hausa TV snapshot classes
ws-nextjs-app/integration/pages/av-embeds/russian/__snapshots__/canonical.test.ts.snap Updates Russian embed snapshot classes
ws-nextjs-app/integration/pages/articles/persianMediaPlayer/__snapshots__/canonical.test.ts.snap Updates Persian media snapshot classes
ws-nextjs-app/integration/pages/articles/afrique/__snapshots__/canonical.test.ts.snap Updates Afrique media snapshot classes
Review details
  • Files reviewed: 5/5 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@alex-magana
alex-magana merged commit 9fe808e into WS-2782-apply-dark-ui-for-topic-pages Sep 11, 2026
15 of 16 checks passed
@alex-magana
alex-magana deleted the WS-2782-update-tests branch September 11, 2026 07:55
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