Skip to content
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.

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

WORLDSERVICE-463: Automated test to ensure local manifest file matches live manifest file for Public Services #12495

Open
wants to merge 7 commits into
base: WORLDSERVICE-420-ps-manifest-file
Choose a base branch
from

Conversation

karinathomasbbc
Copy link
Contributor

@karinathomasbbc karinathomasbbc commented Mar 11, 2025

Resolves JIRA https://jira.dev.bbc.co.uk/browse/WORLDSERVICE-463

Overall changes

Adds an automated test to check that local manifest file matches the live manifest file for PS AMP article pages

Code changes

  • Add a new test for checking the manifest files for PS AMP pages
  • Include this test in the existing amp:validate tests

Testing

  • PR checks

Please note, these will fail until either #12479 or https://github.com/bbc/belfrage/pull/3779 has been deployed to live

@karinathomasbbc karinathomasbbc self-assigned this Mar 11, 2025
Copy link
Contributor

@emilysaffron emilysaffron left a comment

Choose a reason for hiding this comment

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

nice ! assume the tests wont pass here until #12479 is merged right?

@karinathomasbbc
Copy link
Contributor Author

nice ! assume the tests wont pass here until #12479 is merged right?

Correct! We need to wait for the changes to be live (which also includes the Belfrage PR), so this will sit in Blocked until then 😸

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.

4 participants