generated from bcgov/vue-scaffold
-
Notifications
You must be signed in to change notification settings - Fork 2
CCFRI-5962 Portal Cypress Testing (Group Application E2E) #825
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ectly but needs to be optimized
…d for all user paths
…ion; sample config file
…ble happy path selections, updated cypress to newest available version
Group Application E2E running fully without errors, edited config file to increase default timeout to account for browser speed variability Preparing for merging with Main handled PR comments
… adding multiple Facilities, Closures, and adding Partial Closures
… efficiency of bug resolution
…s, finalized e2e script
weskubo-cgi
requested changes
Sep 27, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, the name is fine for but the branch name should be a little more specific. E.g. portal-cypress-group-application-testing
test/cypress-e2e/cypress/e2e/2-group-application/full-e2e.cy.js
Outdated
Show resolved
Hide resolved
test/cypress-e2e/cypress/e2e/2-group-application/full-e2e.cy.js
Outdated
Show resolved
Hide resolved
...press-e2e/cypress/e2e/2-group-application/02-organization-info-group-application-CCFRI.cy.js
Outdated
Show resolved
Hide resolved
...ress-e2e/cypress/e2e/2-group-application/03-organization-info-group-application-ECE-WE.cy.js
Outdated
Show resolved
Hide resolved
...press-e2e/cypress/e2e/2-group-application/02-organization-info-group-application-CCFRI.cy.js
Outdated
Show resolved
Hide resolved
...press-e2e/cypress/e2e/2-group-application/02-organization-info-group-application-CCFRI.cy.js
Outdated
Show resolved
Hide resolved
Please check the SonarQube quality gate as well as there is a failure. |
weskubo-cgi
requested changes
Oct 2, 2025
...press/support/pages/2-group-application-pages/01-organization-info-group-application-ccof.js
Outdated
Show resolved
Hide resolved
...ress/support/pages/2-group-application-pages/02-organization-info-group-application-CCFRI.js
Outdated
Show resolved
Hide resolved
…converting non-reassigned variables to constants. Updated READ.me and example .env file to clarify setup for future team members
|
weskubo-cgi
approved these changes
Oct 2, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Implemented E2E automation of CCOF group application submission within the Portal.
Types of changes
Automation - Includes ability to add a single facility, single full facility closure, and submit an ECE-WE application. E2E script runs through starting a new application -> verifying successful submission & logging out of portal.
Checklist
Further comments
Most recent push includes a significant change to the test structure in order to reduce code duplication, as well as improving adherence to Cypress best practices.
Moving forward, larger e2e tests will be created as Page Objects and added to support > pages. This allows for ease of debugging in the future, if necessary, as well as increased reusability of smaller e2e components as each page object can be called in multiple different e2e tests. This has significantly improved the readability of each e2e test and minimized code duplication.
Much of the scripts have also been edited to reduce getting elements based on class tags or other attributes as these are volatile and noted as bad practices in Cypress. Adjustments have been made to minimize finding elements in this manner.