Skip to content

Migrate homepage e2e to nextjs #13579

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
amoore108 merged 23 commits into from
Jan 5, 2026
Merged

Migrate homepage e2e to nextjs #13579

amoore108 merged 23 commits into from
Jan 5, 2026

Conversation

@amoore108
Copy link
Contributor

@amoore108 amoore108 commented Jan 2, 2026 •
edited
Loading

Resolves JIRA: https://bbc.atlassian.net/browse/WS-1827

Summary

  • Moves Cypress tests from Express app to Next.js app
  • Small refactor to the handleHomepageRoute function to account for invalid service requests

Developer Checklist

  • UX
    • UX Criteria met (visual UX & screenreader UX)
  • Accessibility
    • Accessibility Acceptance Criteria met
    • Accessibility swarm completed
    • Component Health updated
    • P1 accessibility bugs resolved
    • P2/P3 accessibility bugs planned (if not resolved)
  • Security
    • Security issues addressed
    • Threat Model updated
  • Documentation
    • Docs updated (runbook, READMEs)
  • Testing
    • Feature tested on relevant environments
  • Comms
    • Relevant parties notified of changes

Testing

  • Manual Testing required?
    • Local (Ready-For-Test, Local)
    • Test (Ready-For-Test, Test)
    • Preview (Ready-For-Test, Preview)
    • Live (Ready-For-Test, Live)
  • Manual Testing complete?
    • Local
    • Test
    • Preview
    • Live

Additional Testing Steps

  1. List the steps required to test this PR.

Useful Links

MeriemMechri and others added 21 commits January 2, 2026 09:31
@MeriemMechri @amoore108
refactor:migrate cypress home page tests to nextJs
3860735
@MeriemMechri @amoore108
refactor:delete homePage old folder
b38d834
@amoore108
Add bbc-adverts header to homepage tests
d544bc7
@amoore108
Extract service from resolvedUrl
518ad79
@amoore108
Migrate across atiAnalytics/assertions/index file
0ea799f
@amoore108
Merge branch 'latest' into migrate-homepage-e2e-to-nextjs
de84976
@amoore108
Merge branch 'latest' into migrate-homepage-e2e-to-nextjs
1fa8589
@amoore108
Set variant in test config
d2f003e
@amoore108
Try with testIsolation
c4b4c4d
@amoore108
Revert "Try with testIsolation"
3d9927e 
This reverts commit c4b4c4d.
@amoore108
try page-type header?
8bd5268
@amoore108
Test forcing page reload
758c573
@amoore108
Try changing ordering of tests
bea7b08
@amoore108
Update urlValidationTest.ts
52d1ce2
@amoore108
Remove next from express cypress config
5ac7c23
@amoore108
Revert "Remove next from express cypress config"
964b630 
This reverts commit 5ac7c23.
@amoore108
Try with .find
c194c9f
@amoore108
Update urlValidationTest.ts
1ed8f8b
@amoore108
Update urlValidationTest.ts
64acac4
@amoore108
Update urlValidationTest.ts
16d7162
@amoore108
Merge branch 'latest' into migrate-homepage-e2e-to-nextjs
e3750d7
@amoore108 amoore108 marked this pull request as ready for review January 5, 2026 09:47
@amoore108 amoore108 merged commit 9d56d51 into latest Jan 5, 2026
13 checks passed
@amoore108 amoore108 deleted the migrate-homepage-e2e-to-nextjs branch January 5, 2026 11:40
@amoore108 amoore108 mentioned this pull request Jan 5, 2026
Closed
27 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@holchris holchris holchris approved these changes

@Nabeel1276 Nabeel1276 Nabeel1276 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@amoore108 @holchris @Nabeel1276 @MeriemMechri