Removing app test from workspace setup test #16
Merged
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
This PR removes the app e2e testing functionality of workspace setup. The original intent of this was to run workspace setup, which clones the repos and installs all dependencies. Then it cd's into the app repo spins up the app on the github runner then runs tests. It worked fine for a while but we're getting errors when running and a failing test is not really a failure of the workspace setup job. It makes things more complicated and confusing than its worth. This just removes that and simplifies things
Related ticket(s)
How to test
on merge to main this should kick off a test and we'll see :)
Notes
Pre-review checklist
Pre-merge checklist
Review
Security
If either of the following are true, notify the team's ISSO (Information System Security Officer).