Skip to content

Commit

Permalink
fix acceptance tests by adding missing var in parameter (#376)
Browse files Browse the repository at this point in the history
* fix acceptance tests by adding missing var in parameter

* adding in the fix for the build manifest as well
  • Loading branch information
rcgottlieb authored Sep 17, 2024
1 parent 634abf1 commit 750d1ad
Show file tree
Hide file tree
Showing 6 changed files with 11 additions and 10 deletions.
Empty file modified ci/build-manifest.sh
100755 → 100644
Empty file.
5 changes: 3 additions & 2 deletions ci/run-smoke-test-rotate-creds.sh
100755 → 100644
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ cf delete-service -f "rds-smoke-tests-db-rotate-creds-$SERVICE_PLAN"

# change into the directory and push the app without starting it.
pushd aws-db-test/databases/aws-rds
cf push "smoke-tests-db-rotate-creds-${SERVICE_PLAN}" -f manifest.yml --no-start
cf push "smoke-tests-db-rotate-creds-${SERVICE_PLAN}" -f manifest.yml --var rds-service="rds-smoke-tests-db-rotate-creds-$SERVICE_PLAN" --no-start

# set some variables that it needs
cf set-env "smoke-tests-db-rotate-creds-${SERVICE_PLAN}" DB_TYPE "${SERVICE_PLAN}"
Expand All @@ -43,7 +43,7 @@ while true; do
done

# wait for the app to start. if the app starts, it's passed the smoke test.
cf push "smoke-tests-db-rotate-creds-${SERVICE_PLAN}"
cf push "smoke-tests-db-rotate-creds-${SERVICE_PLAN}" --var rds-service="rds-smoke-tests-db-rotate-creds-$SERVICE_PLAN"

# Rotate creds
cf update-service "rds-smoke-tests-db-rotate-creds-$SERVICE_PLAN" -c '{"rotate_credentials": true}'
Expand All @@ -69,6 +69,7 @@ cf restage "smoke-tests-db-rotate-creds-${SERVICE_PLAN}"
# Restart app - if it succeeds, then smoke tests have passed with new credentials
cf restart "smoke-tests-db-rotate-creds-${SERVICE_PLAN}"


# Clean up app and service
cf delete -f "smoke-tests-db-rotate-creds-$SERVICE_PLAN"
cf delete-service -f "rds-smoke-tests-db-rotate-creds-$SERVICE_PLAN"
4 changes: 2 additions & 2 deletions ci/run-smoke-tests-db-updates.sh
100755 → 100644
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ cf delete-service -f "rds-smoke-tests-db-update-$SERVICE_PLAN"

# change into the directory and push the app without starting it.
pushd aws-db-test/databases/aws-rds
cf push "smoke-tests-db-update-${SERVICE_PLAN}" -f manifest.yml --no-start
cf push "smoke-tests-db-update-${SERVICE_PLAN}" -f manifest.yml --var rds-service="rds-smoke-tests-update-$SERVICE_PLAN" --no-start

# set some variables that it needs
cf set-env "smoke-tests-db-update-${SERVICE_PLAN}" DB_TYPE "${SERVICE_PLAN}"
Expand All @@ -50,7 +50,7 @@ while true; do
done

# wait for the app to start. if the app starts, it's passed the smoke test.
cf push "smoke-tests-db-update-${SERVICE_PLAN}"
cf push "smoke-tests-db-update-${SERVICE_PLAN}" --var rds-service="rds-smoke-tests-db-update-$SERVICE_PLAN"

# Update service
cf update-service "rds-smoke-tests-db-update-$SERVICE_PLAN" -p "$NEW_SERVICE_PLAN"
Expand Down
4 changes: 2 additions & 2 deletions ci/run-smoke-tests-db-version.sh
100755 → 100644
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ cf delete-service -f "rds-smoke-tests-db-version-$SERVICE_PLAN"

# change into the directory and push the app without starting it.
pushd aws-db-test/databases/aws-rds
cf push "smoke-tests-db-version-${SERVICE_PLAN}" -f manifest.yml --no-start
cf push "smoke-tests-db-version-${SERVICE_PLAN}" -f manifest.yml --var rds-service="rds-smoke-tests-db-version-$SERVICE_PLAN" --no-start

# set some variables that it needs
cf set-env "smoke-tests-db-version-${SERVICE_PLAN}" DB_TYPE "${SERVICE_PLAN}"
Expand All @@ -49,7 +49,7 @@ while true; do
done

# wait for the app to start. if the app starts, it's passed the smoke test.
cf push "smoke-tests-db-version-${SERVICE_PLAN}"
cf push "smoke-tests-db-version-${SERVICE_PLAN}" --var rds-service="rds-smoke-tests-db-version-$SERVICE_PLAN"

# Clean up app and service
cf delete -f "smoke-tests-db-version-$SERVICE_PLAN"
Expand Down
4 changes: 2 additions & 2 deletions ci/run-smoke-tests-update-storage.sh
100755 → 100644
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ cf delete-service -f "rds-smoke-tests-db-update-storage-$SERVICE_PLAN"

# change into the directory and push the app without starting it.
pushd aws-db-test/databases/aws-rds
cf push "smoke-tests-db-update-storage-${SERVICE_PLAN}" -f manifest.yml --no-start
cf push "smoke-tests-db-update-storage-${SERVICE_PLAN}" -f manifest.yml --var rds-service="rds-smoke-tests-db-update-storage-$SERVICE_PLAN" --no-start

# set some variables that it needs
cf set-env "smoke-tests-db-update-storage-${SERVICE_PLAN}" DB_TYPE "${SERVICE_PLAN}"
Expand All @@ -50,7 +50,7 @@ while true; do
done

# wait for the app to start. if the app starts, it's passed the smoke test.
cf push "smoke-tests-db-update-storage-${SERVICE_PLAN}"
cf push "smoke-tests-db-update-storage-${SERVICE_PLAN}" --var rds-service="rds-smoke-tests-db-update-storage-$SERVICE_PLAN"

# Update storage size
cf update-service "rds-smoke-tests-db-update-storage-$SERVICE_PLAN" -c '{"storage": 25}'
Expand Down
4 changes: 2 additions & 2 deletions ci/run-smoke-tests.sh
100755 → 100644
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ cf delete-service -f "rds-smoke-tests-$SERVICE_PLAN"

# change into the directory and push the app without starting it.
pushd aws-db-test/databases/aws-rds
cf push "smoke-tests-${SERVICE_PLAN}" -f manifest.yml --no-start
cf push "smoke-tests-${SERVICE_PLAN}" -f manifest.yml --var rds-service="rds-smoke-tests-$SERVICE_PLAN" --no-start

# set some variables that it needs
cf set-env "smoke-tests-${SERVICE_PLAN}" DB_TYPE "${SERVICE_PLAN}"
Expand All @@ -37,7 +37,7 @@ while true; do
done

# wait for the app to start. if the app starts, it's passed the smoke test.
cf push "smoke-tests-${SERVICE_PLAN}"
cf push "smoke-tests-${SERVICE_PLAN}" --var rds-service="rds-smoke-tests-$SERVICE_PLAN"

# Clean up app and service
cf delete -f "smoke-tests-$SERVICE_PLAN"
Expand Down

0 comments on commit 750d1ad

Please sign in to comment.