Skip to content

Implement test isolation in Cypress ATI Analytics tests#12522

Merged
karinathomasbbc merged 3 commits intolatestfrom
cypress-test-isolation
Mar 18, 2025
Merged

Implement test isolation in Cypress ATI Analytics tests#12522
karinathomasbbc merged 3 commits intolatestfrom
cypress-test-isolation

Conversation

@karinathomasbbc
Copy link
Contributor

@karinathomasbbc karinathomasbbc commented Mar 18, 2025

Overall changes

Implement test isolation in Cypress per the docs: https://docs.cypress.io/app/core-concepts/test-isolation

Code changes

  • Enable testIsolation for ATI Analytics tests
  • Visit the test path in each test (as the default page is about:blank)
  • Remove unnecessary click back to the previous page in each assertion

Testing

  • with simorgh running (yarn build; yarn start) -> CYPRESS_APP_ENV=local yarn cypress run --spec './cypress/e2e/specialFeatures/atiAnalytics/index.cy.js'
  • CYPRESS_APP_ENV=test yarn cypress run --spec './cypress/e2e/specialFeatures/atiAnalytics/index.cy.js'
  • CYPRESS_APP_ENV=live yarn cypress run --spec './cypress/e2e/specialFeatures/atiAnalytics/index.cy.js'

@karinathomasbbc karinathomasbbc self-assigned this Mar 18, 2025
@karinathomasbbc karinathomasbbc changed the title Implement test isolation in Cypress Implement test isolation in Cypress ATI Analytics tests Mar 18, 2025
@karinathomasbbc karinathomasbbc merged commit c2620c9 into latest Mar 18, 2025
11 checks passed
@karinathomasbbc karinathomasbbc deleted the cypress-test-isolation branch March 18, 2025 11:27
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.

3 participants