Swapping getSecret priority

- Swaps priority of getSecrets function to match getEnv, so that pipeline always takes precedence
- Updates tests to ensure they run

Fixes: #112

Author: JoeColeman95

s3secrets-helper/secrets/secrets.go

@@ -146,8 +146,8 @@ func getSecrets(conf Config, results chan<- getResult) {
 	}...)
 
 	prefixes := []string{
-		conf.Prefix + "/secret-files",
 		"secret-files",
+		conf.Prefix + "/secret-files",
 	}
 
 	conf.Logger.Printf("Checking S3 for secret-files")

s3secrets-helper/secrets/secrets_test.go

@@ -160,12 +160,12 @@ func TestRun(t *testing.T) {
 		// because git-credentials were found:
 		// (wrap in double quotes so that bash eval doesn't consume the inner single quote.
 		`GIT_CONFIG_PARAMETERS="` + gitCredentialHelpers + `"`,
+		"ORG_SERVICE_TOKEN='org service token'",
 		"BUILDKITE_ACCESS_KEY='buildkite access key'",
 		"DATABASE_SECRET='database secret'",
 		"EXTERNAL_API_SECRET_KEY='external api secret'",
 		"PRIVILEGED_PASSWORD='privileged password'",
 		"SERVICE_TOKEN='service token'",
-		"ORG_SERVICE_TOKEN='org service token'",
 	}, "\n") + "\n"
 
 	if actual := envSink.String(); expected != actual {

tests/environment.bats

@@ -1,6 +1,6 @@
 #!/usr/bin/env bats
 
-load '/usr/local/lib/bats/load.bash'
+load '/usr/lib/bats/load.bash'
 
 # export AWS_STUB_DEBUG=/dev/tty
 # export SSH_ADD_STUB_DEBUG=/dev/tty