Skip to content

Fcrm 6803 add p1 tests to results and next steps#768

Merged
garydawsonDEFRA merged 8 commits intodevelopmentfrom
FCRM-6803-AddP1TestsToResultsAndNextSteps
Apr 28, 2026
Merged

Fcrm 6803 add p1 tests to results and next steps#768
garydawsonDEFRA merged 8 commits intodevelopmentfrom
FCRM-6803-AddP1TestsToResultsAndNextSteps

Conversation

@garydawsonDEFRA
Copy link
Copy Markdown
Collaborator

Implemented a P1 PDF download testing setup. This included adding a PDFDriver file to handle download events and responses, parse PDF content, and perform assertions and adding coverage for Results and Next Steps Spec files across flood zones 1–3.

Updated README, added PDF downloads to the git-ignore file.

Other additional changes include; removing confirmation page test from check your details spec file and making E2E test run in Pre only.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 20, 2026

Test Results

68 tests   68 ✅  55s ⏱️
14 suites   0 💤
 1 files     0 ❌

Results for commit a19cae8.

♻️ This comment has been updated with latest results.

Comment thread e2e/tests/pages/next-steps-page.spec.js Outdated
Comment thread e2e/tests/pages/results-page.spec.js Outdated
Comment thread e2e/tests/pages/results-page.spec.js Outdated
Comment thread e2e/test-runner-api/pdf-driver.js Outdated
Comment thread e2e/test-runner-api/pdf-driver.js Outdated
Comment thread e2e/test-runner-api/pdf-driver.js Outdated
Comment thread e2e/tests/pages/results-page.spec.js Outdated
Comment thread e2e/test-runner-api/pdf-driver.js
Copy link
Copy Markdown
Contributor

@RyonLeightonDEFRA RyonLeightonDEFRA left a comment

Choose a reason for hiding this comment

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

Added comments and new branch as per teams chat

* pdf updates

Co-authored-by: Copilot <copilot@github.com>

* removed test.describe.only from results spec file

---------

Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Gary Dawson <gary.dawson@defra.gov.uk>
@RyonLeightonDEFRA RyonLeightonDEFRA self-requested a review April 28, 2026 09:41
@sonarqubecloud
Copy link
Copy Markdown

@garydawsonDEFRA garydawsonDEFRA merged commit 8440922 into development Apr 28, 2026
3 checks passed
@garydawsonDEFRA garydawsonDEFRA deleted the FCRM-6803-AddP1TestsToResultsAndNextSteps branch April 28, 2026 10:06
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