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

Cypress integration test workflow #32279

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

IGallupSoCo
Copy link
Contributor

@IGallupSoCo IGallupSoCo commented Oct 4, 2024

Are you removing, renaming or moving a folder in this PR?

  • No, I'm not changing any folders (skip to TeamSites and delete the rest of this section)
  • Yes, I'm removing, renaming or moving a folder

Summary

  • A manual workflow has been created to support the troubleshooting of CI failures in a more time-efficient way. The user provides inputs for the relevant product directory within src/applications and specifies the branch. The workflow then takes those inputs and runs any Cypress specs that can be found anywhere within the specified directory. The workflow runs are considerably simpler and shorter than in vets-website CI, and should enable engineers to identify patterns behind e2e test failures more quickly, and to rapidly test any attempted fixes as they are added to a given branch.

Related issue(s)

What areas of the site does it impact?

This adds a manual workflow and one supporting shell file.

Acceptance criteria

Quality Assurance & Testing

  • No sensitive information (i.e. PII/credentials/internal URLs/etc.) is captured in logging, hardcoded, or specs
  • Linting warnings have been addressed
  • Documentation has been updated (link to design doc)

@IGallupSoCo IGallupSoCo self-assigned this Oct 4, 2024
@IGallupSoCo IGallupSoCo requested a review from a team as a code owner October 4, 2024 19:41
@va-vfs-bot va-vfs-bot temporarily deployed to master/cypress-integration-test-workflow/main October 4, 2024 19:59 Inactive
Copy link
Contributor

@rmessina1010 rmessina1010 left a comment

Choose a reason for hiding this comment

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

LGTM

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