Use env vars to pass cf credentials in tests instead of janky echo mo…

…dule

.github/workflows/test.yml

@@ -19,8 +19,8 @@ jobs:
 
       - name: terraform test ${{ matrix.module }}
         uses: dflook/terraform-test@v1
+        env:
+          CF_USER: ${{ secrets.CF_USER }}
+          CF_PASSWORD: ${{ secrets.CF_PASSWORD }}
         with:
           path: ${{ matrix.module }}
-          variables: |
-            cf_user = "${{ secrets.CF_USER }}"
-            cf_password = "${{ secrets.CF_PASSWORD }}"

README.md

@@ -145,10 +145,7 @@ module "egress_space" {
 
 [Terraform tests](https://developer.hashicorp.com/terraform/language/tests) are in progress of being written. To run for any module with a `tests` directory:
 
+1. Set `CF_USER` and `CF_PASSWORD` env variables with SpaceDeployer credentials that can access the space(s) being used for tests
 1. cd to module root. Example: `cd s3`
 1. Run `terraform init`
-1. Run `terraform test -var cf_user="SERVICE_DEPLOYER_USERNAME" -var cf_password="SERVICE_DEPLOYER_PASSWORD"`
-
-### tf-cg-test-support
-
-This module is used to help setup the cloudfoundry provider for each test. It is not a useful module on its own
+1. Run `terraform test`

s3/tests/creation.tftest.hcl

@@ -1,7 +1,6 @@
 provider "cloudfoundry" {
-  api_url  = "https://api.fr.cloud.gov"
-  user     = run.setup.cf_user
-  password = run.setup.cf_password
+  api_url = "https://api.fr.cloud.gov"
+  # cf_user and cf_password are passed in via CF_USER and CF_PASSWORD env vars
 }
 
 variables {
@@ -11,12 +10,6 @@ variables {
   name          = "terraform-cloudgov-s3-test"
 }
 
-run "setup" {
-  module {
-    source = "../tf-cg-test-support"
-  }
-}
-
 run "test_creation" {
   assert {
     condition     = can(regex("^\\w{8}-\\w{4}-\\w{4}-\\w{4}-\\w{12}$", output.bucket_id))