Skip to content
This repository was archived by the owner on Aug 4, 2025. It is now read-only.

Commit 904b031

Browse files
committed
Prefer Pact Broker in human-friendly text.
1 parent 2106d46 commit 904b031

File tree

3 files changed

+6
-7
lines changed

3 files changed

+6
-7
lines changed

.github/workflows/verify_consumer_pacts.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ name: Verify consumer pacts
1111
# 2. Provider makes a change (runs verification tests against ALL DEPLOYED consumer pact versions and publishes corresponding verification results)
1212
#
1313
#
14-
# The workflow requires the following Pact broker credentials:
14+
# The workflow requires the following Pact Broker credentials:
1515
# - PACT_BROKER_USERNAME - the Pact Broker username
1616
# - PACT_BROKER_PASSWORD - the Pact Broker password
1717
# They are managed by Atlantis and were added to Terraform here:

README.md

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -698,19 +698,18 @@ The integration tests live in the `integration` project. Consult the integration
698698
In the early days of the project, there were JUnit-based integration tests. We are in
699699
process of migrating them to Test Runner.
700700

701-
702701
### Pact Tests
703702

704703
Pact testing ensures workspace manager's APIs are compatible with the assumptions made
705704
by its clients and that workspace manager's assumptions about its dependency APIs are
706705
also correct.
707706

708-
Pact testing involves interacting with a pact-broker which requires a little setup.
707+
Pact testing involves interacting with Pact Broker which requires a little setup.
709708

710709
To run pact tests locally:
711710

712711
```
713-
# Get pactbroker credentials
712+
# Get Pact Broker credentials
714713
./service/src/test/render-pact-configs.sh
715714
# Reload your environment variables, e.g. src ~/.zshrc
716715
@@ -722,10 +721,10 @@ export PACT_BROKER_PASSWORD=$(cat /tmp/pact-ro-password.key)
722721
```
723722

724723
If you're working on adding new pacts and making local changes to them, it might be helpful to use a local
725-
pactbroker instead. To set up a local pactbroker, see:
724+
Pact Broker instead. To set up a local Pact Broker, see:
726725
[Contract Test Local Development](https://broadworkbench.atlassian.net/wiki/spaces/IRT/pages/2829680649/Contract+Test+Local+Development).
727726

728-
Once you have a local pactbroker, you can override the `PACT_BROKER_URL` environment variable:
727+
Once you have a local Pact Broker, you can override the `PACT_BROKER_URL` environment variable:
729728

730729
```
731730
PACT_BROKER_URL=http://localhost:9292 ./gradlew verifyPacts

service/src/test/java/bio/terra/workspace/pact/WdsContractVerificationTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@
4141

4242
@ActiveProfiles({
4343
"unit-test" /* disable some unnecessary dependencies that would otherwise require mocking */,
44-
"pact-test" /* set pact broker URL */
44+
"pact-test" /* set Pact Broker URL */
4545
})
4646
@Tag("pact-verification")
4747
@Provider("workspacemanager") // should match the terra chart name

0 commit comments

Comments
 (0)