WS-3085-Add appropriate test coverage for /watch and /listen URLs - #14343
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds new integration test coverage (with snapshots and fixture data) for additional canonical /watch and /listen routes in the Next.js app, extending the existing page-level integration test suite.
Changes:
- Added canonical integration tests + snapshots for Tamil
/watchand/listenroutes. - Added canonical integration test + snapshots for Gahuza
/watch(but currently targets an/articlesURL). - Added new Optimo fixture JSON for a Gahuza article used by the new integration test.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| ws-nextjs-app/integration/pages/watch/tamil/canonical.test.ts | Adds Tamil canonical test coverage for the /watch route using existing shared canonical + media-player suites. |
| ws-nextjs-app/integration/pages/watch/tamil/snapshots/canonical.test.ts.snap | Stores expected rendered output snapshots for the Tamil /watch canonical suite. |
| ws-nextjs-app/integration/pages/watch/gahuza/canonical.test.ts | Adds Gahuza canonical test coverage under /watch (currently configured to hit an /articles pathname). |
| ws-nextjs-app/integration/pages/watch/gahuza/snapshots/canonical.test.ts.snap | Stores expected rendered output snapshots for the Gahuza watch canonical suite (currently asserting /articles canonical URL). |
| ws-nextjs-app/integration/pages/listen/tamil/canonical.test.ts | Adds Tamil canonical test coverage for the /listen route using existing shared canonical + audio media-player suites. |
| ws-nextjs-app/integration/pages/listen/tamil/snapshots/canonical.test.ts.snap | Stores expected rendered output snapshots for the Tamil /listen canonical suite. |
| data/gahuza/articles/clyw2p2x138o.json | Adds fixture data required to run the new Gahuza integration test. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Nabeel1276
changed the title
alex-magana
reviewed
Aug 25, 2026
…c/simorgh into WS-3085-add-appropriate-test-coverage
pvaliani
approved these changes
Aug 26, 2026
pvaliani
reviewed
Aug 26, 2026
alex-magana
approved these changes
Aug 26, 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.
Resolves JIRA: https://bbc.atlassian.net/browse/WS-3085
Summary
This PR adds integration test coverage for the /watch and /listen URL routes, ensuring they behave as expected. End-to-end coverage will be added once the new routing changes are live. A comprehensive set of as unit tests were covered in #14315
Code changes
Testing
Useful Links