Skip to content

Commit

Permalink
Ensure cf push of the example-service-instances-api app is checked
Browse files Browse the repository at this point in the history
This failed because the app manifest was hardcoded with GOVERSION=1.21
which was no longer supported by the cloudfoundry environment.
Unfortunately the test proceeded anyway and failed later in a way that
was harder to troubleshoot.

Now the feature flags tests asserts that cf push actually succeeds - or
at least cf push exits non-zero and flags a better error message
up-front.
  • Loading branch information
abg committed Oct 1, 2024
1 parent 6bc03b1 commit 16095e2
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions system_tests/feature_flags/feature_flags_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -81,13 +81,13 @@ var _ = Describe("FeatureFlags", func() {
siAPIUsername := "siapi"
siAPIPassword := "siapipass"

cf.Cf("push",
Expect(cf.Cf("push",
"-p", os.Getenv("SI_API_PATH"),
"-f", os.Getenv("SI_API_PATH")+"/manifest.yml",
"--var", "app_name="+appName,
"--var", "username="+siAPIUsername,
"--var", "password="+siAPIPassword,
)
)).To(gexec.Exit(0), `cf push example-service-instances-api app failed!`)

brokerInfo = bosh.DeployBroker(
"-feature-flag-"+uniqueID,
Expand Down

0 comments on commit 16095e2

Please sign in to comment.