Skip to content

[BDMS-264] Patch CI/CD Cypress Testing#132

Merged
TylerAdamMartinez merged 35 commits intostagingfrom
TAM-patch-CI-CD
Nov 5, 2025
Merged

[BDMS-264] Patch CI/CD Cypress Testing#132
TylerAdamMartinez merged 35 commits intostagingfrom
TAM-patch-CI-CD

Conversation

@TylerAdamMartinez
Copy link
Copy Markdown
Contributor

Why

This PR addresses the following problem/context:

  • To keep the frontend and backend consistent, we need to use the latest version of the backend while testing.

How

Implementation summary - the following was changed/added/removed:

  • During CI/CD testing, we will spin up a local version of the backend on the staging branch and use that version while testing.

Notes

Any special considerations, workarounds, or follow-up work to note?

  • THESE ARE BREAKING CHANGES TO THE TESTS SCRIPTS
  • Also expect to see a lot of pipelines failing before we get a successful implementation of this.

@TylerAdamMartinez TylerAdamMartinez changed the title NO TICKET patch ci cd [BDMS-264] Patch ci cd Nov 4, 2025
@TylerAdamMartinez TylerAdamMartinez changed the title [BDMS-264] Patch ci cd [BDMS-264] Patch CI/CD Cypress Testing Nov 5, 2025
@TylerAdamMartinez TylerAdamMartinez marked this pull request as ready for review November 5, 2025 14:31
Copy link
Copy Markdown
Collaborator

@chasetmartin chasetmartin left a comment

Choose a reason for hiding this comment

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

This looks good to me, nice improvements to the settings file to determine test type. I take it once your other PR is merged you plan to change the ref to staging on this PR?

- name: Save build folder
uses: actions/upload-artifact@v4
with:
name: build
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Yeah I was following the cypress docs example with this build and upload artifact step, but it seems redundant to me so I like that you removed it.

@TylerAdamMartinez
Copy link
Copy Markdown
Contributor Author

This looks good to me, nice improvements to the settings file to determine test type. I take it once your other PR is merged you plan to change the ref to staging on this PR?

Yes, of course.

@TylerAdamMartinez TylerAdamMartinez merged commit df4cd3a into staging Nov 5, 2025
3 checks passed
@TylerAdamMartinez TylerAdamMartinez deleted the TAM-patch-CI-CD branch November 5, 2025 18:17
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