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.

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

Feature/e2e tests conversion to playwright #486

Draft
wants to merge 40 commits into
base: release/02-Dec-24-R1
Choose a base branch
from
Draft

Feature/e2e tests conversion to playwright #486

wants to merge 40 commits into from
+740 −26

Conversation

DanielClarkeEducation
Copy link
Contributor

Description

Please include a summary of the changes.

Ticket number (if applicable)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

Screenshots

Please include screenshots if applicable.

Checklist:

  • My code follows the standards used within this project
  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests (Unit, E2E) that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

DanielClarkeEducation and others added 30 commits December 2, 2024 13:46
@DanielClarkeEducation
enabled webkit smoke tests
eaae6af
@DanielClarkeEducation
Initialised Playwright for E2E tests plus updated README with details…
cd23035 
… on Playwright
@DanielClarkeEducation
release branch merge
f3e9183
@DanielClarkeEducation
Added new back button tests + new action and modification to PR workflow
eac53a8
@DanielClarkeEducation
fixed GitHub action
ecf297c
@DanielClarkeEducation
Added base url to playwright config
e032e2a
@DanielClarkeEducation
change to back button tests
58a06d0
@DanielClarkeEducation
fix for checking home page
a1694e5
@DanielClarkeEducation
test
027e399
@DanielClarkeEducation
test
4767019
@DanielClarkeEducation
Merge branch 'release/02-Dec-24-R1' into feature/e2e-tests-conversion…
45e1afb 
…-to-playwright
@ThomasWhittington
test: added more e2e tests in playwright
651c570
@sam-c-dfe
Merge branch 'release/02-Dec-24-R1' into feature/e2e-tests-conversion…
7e14903 
…-to-playwright
@sam-c-dfe
Added domain to env variables
2c6f471
@sam-c-dfe
Updated url
7248f53
@sam-c-dfe
Wrapped url in quotes
a0adb8c
@sam-c-dfe
Added tracing
7784c98
@sam-c-dfe
Updated action
f6607e0
@sam-c-dfe
Updated config and added a print to see what the page url is to help …
997dfd8 
…debug the failures
@sam-c-dfe
Amended url
a83d6aa
@sam-c-dfe
Extended timeout
8b88416
@sam-c-dfe
Changed the domain
ef5f4cb
@sam-c-dfe
Switched to yarn to try fix running issue
185db13
@sam-c-dfe
Changed node version
4ea89cc
@sam-c-dfe
Changed runner
2b7cb06
@sam-c-dfe
Changed the runner
1e154e1
@sam-c-dfe
Changed the trace type
4fbc9bd
@sam-c-dfe
added page title to the output
3daafbe
@sam-c-dfe
Added filter to shorten the number of tests run for now
50aac87
@sam-c-dfe
Added waituntil
4e450bf
@github-actions GitHub Actions
Copy link
Contributor

Code Coverage

Package Line Rate Branch Rate Health
Dfe.EarlyYearsQualification.Content 98% 90% ✔
Dfe.EarlyYearsQualification.Web 96% 90% ✔
Dfe.EarlyYearsQualification.Mock 100% 98% ✔
Summary 97% (4127 / 4235) 91% (686 / 754) ✔

Minimum allowed line rate is 90%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@DanielClarkeEducation @ThomasWhittington @sam-c-dfe