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

Integration tests #502

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

Conversation

divolgin
Copy link
Member

@divolgin divolgin commented Jan 25, 2025

This replaces integration tests with tests that use a mock web server.

  1. Adds tests for all replicated api subcommands.
  2. Replaces tests for all replicated app subcommands with new tests.

Other changes:

  1. Removed e2e step. The e2e docker file did not actually do anything.
  2. Removed a github actions workflow file (working.yaml) that ran exact same steps as main.yaml
  3. Updated pact env variables to be only set on main branch, so pact isn't published from dev branches
  4. Changed unit, pact, and integration tests to run in parallel using separate GHA jobs

@divolgin divolgin force-pushed the divolgin/sc-114365/add-tests-for-replicated-cli-to-improve-release branch from db8e68f to b1e3219 Compare January 25, 2025 19:26
@divolgin divolgin force-pushed the divolgin/sc-114365/add-tests-for-replicated-cli-to-improve-release branch 3 times, most recently from 13f5b3c to 69c9f0d Compare January 25, 2025 19:59
@divolgin divolgin force-pushed the divolgin/sc-114365/add-tests-for-replicated-cli-to-improve-release branch from 69c9f0d to a4b4d44 Compare January 25, 2025 20:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant