Skip to content

test(e2e): tolerate legacy-environment content on deploy-test - #179

Merged
libredee merged 1 commit into
mainfrom
dee/deploy-test-legacy-env-compat
Aug 3, 2026
Merged

test(e2e): tolerate legacy-environment content on deploy-test#179
libredee merged 1 commit into
mainfrom
dee/deploy-test-legacy-env-compat

Conversation

@libredee

@libredee libredee commented Aug 3, 2026

Copy link
Copy Markdown
Contributor
  • The post-merge deploy-test UI tests run against the deployed test sites, which serve the master Contentful environment (legacy content model) — a combination PR CI (which uses development) never exercises. Two tests failed across all browsers/brands on run 30815853217.
  • SEO test: the legacy SEO Meta Data entry authors OpenGraph title/description separately, while the new model mirrors metaTitle/metaDesc. The test now branches on the observed shape; the legacy arm is marked for removal after the Contentful environment switch.
  • i18n test: accepts both dropdown labels — Select language (redesigned Code.org footer) and Language selection dropdown (MUI footer, used by csforall permanently and by corporate on the legacy environment).
  • Verified all assertions against both live test sites, including the ar language switch through the legacy dropdown.

The test/production sites still serve the master Contentful environment
(legacy SEO Meta Data entry, MUI footer), which PR CI never exercises.
Accept both content shapes; the SEO legacy arm goes away after the
environment switch, the dropdown label alternation stays while csforall
uses the MUI footer.
@libredee
libredee requested a review from stephenliang August 3, 2026 15:11

@cat5inthecradle cat5inthecradle 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.

LGTM

@libredee
libredee merged commit 185884e into main Aug 3, 2026
30 of 38 checks passed
@libredee
libredee removed the request for review from stephenliang August 3, 2026 15:26
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