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.

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

add cypress tests back to post 911 #33258

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import Timeouts from 'platform/testing/e2e/timeouts';

Check warning on line 1 in src/applications/post-911-gib-status/tests/01-authed.cypress.spec.js

View workflow job for this annotation

GitHub Actions / App Isolation Annotations

Staged Continuous Deployment App Isolation Conflict

*WARNING* This PR contains changes related to an application that is currently not isolated. As of Feb 3, 2025 deployment may no longer be possible for apps that are not isolated. Please isolate this app from other directories in 'src/applications' to prevent future deployment issues. More information on your app's status can be seen here: https://department-of-veterans-affairs.github.io/veteran-facing-services-tools/frontend-support-dashboard/cross-app-import-report Please reach out to Frontend Platform Support with any questions.
import enrollmentData from './fixtures/mocks/enrollmentData.json';
import backendStatus from './fixtures/mocks/backendStatus.json';

Expand Down Expand Up @@ -27,13 +27,13 @@
},
).click();

// cy.get('#gibs-full-name').should('contain', 'First Last');
cy.get('#gibs-full-name').should('contain', 'First Last');

// cy.get('#print-button').click();
// cy.get('.print-status', { timeout: Timeouts.slow }).should('be.visible');
// cy.injectAxeThenAxeCheck();
// cy.get('.section-header')
// .should('contain', 'Post-9/11 GI Bill')
// .and('contain', 'Statement of Benefits');
cy.get('#print-button').click();
cy.get('.print-status', { timeout: Timeouts.slow }).should('be.visible');
cy.injectAxeThenAxeCheck();
cy.get('.section-header')
.should('contain', 'Post-9/11 GI Bill')
.and('contain', 'Statement of Benefits');
});
});
2 changes: 1 addition & 1 deletion src/platform/user/tests/unauthed-flows.cypress.spec.js
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
describe('Unauthed User Flow Test', () => {
it('Provides the correct experience', () => {

Check warning on line 2 in src/platform/user/tests/unauthed-flows.cypress.spec.js

View workflow job for this annotation

GitHub Actions / Linting (Files Changed)

src/platform/user/tests/unauthed-flows.cypress.spec.js:2:3:Cypress E2E tests must include at least one axeCheck call. Documentation for adding checks and understanding errors can be found here: https://depo-platform-documentation.scrollhelp.site/developer-docs/A11y-Testing.1935409178.html

Check warning on line 2 in src/platform/user/tests/unauthed-flows.cypress.spec.js

View workflow job for this annotation

GitHub Actions / Linting (Files Changed)

src/platform/user/tests/unauthed-flows.cypress.spec.js:2:3:Cypress E2E tests must include at least one axeCheck call. Documentation for adding checks and understanding errors can be found here: https://depo-platform-documentation.scrollhelp.site/developer-docs/A11y-Testing.1935409178.html
const appPaths = [
// While the page is in maintenance, it doesn't need authed
// Add Below code after content build code is merged and this code is deployed
// '/education/gi-bill/post-9-11/ch-33-benefit/status',
'/education/check-remaining-post-9-11-gi-bill-benefits/status',
'/records/download-va-letters/letters',
'/track-claims',
];
Expand Down
Loading