Skip to content

Commit

Permalink
refactor: expose username and password env vars
Browse files Browse the repository at this point in the history
  • Loading branch information
Philip Germanov committed Oct 10, 2024
1 parent 84cf942 commit d4cf486
Show file tree
Hide file tree
Showing 3 changed files with 36 additions and 18 deletions.
11 changes: 7 additions & 4 deletions cmd/npmExecuteTests.go
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ func runNpmExecuteTests(config *npmExecuteTestsOptions, c command.ExecRunner) er
}

for _, app := range apps {
credentialsToEnv(app.Username, app.Password, config.CredentialsEnvVarPrefix, c)
credentialsToEnv(app.Username, app.Password, config.UsernameEnvVar, config.PasswordEnvVar, c)
err := runTestForUrl(app.URL, config, c)
if err != nil {
return err
Expand All @@ -66,7 +66,7 @@ func runNpmExecuteTests(config *npmExecuteTestsOptions, c command.ExecRunner) er

username := config.VaultMetadata["username"].(string)
password := config.VaultMetadata["password"].(string)
credentialsToEnv(username, password, config.CredentialsEnvVarPrefix, c)
credentialsToEnv(username, password, config.UsernameEnvVar, config.PasswordEnvVar, c)
if err := runTestForUrl(config.BaseURL, config, c); err != nil {
return err
}
Expand All @@ -86,6 +86,9 @@ func runTestForUrl(url string, config *npmExecuteTestsOptions, command command.E
return nil
}

func credentialsToEnv(username, password, prefix string, c command.ExecRunner) {
c.SetEnv([]string{prefix + "_username=" + username, prefix + "_password=" + password})
func credentialsToEnv(username, password, usernameEnv, passwordEnv string, c command.ExecRunner) {
if username == "" || password == "" {
return
}
c.SetEnv([]string{usernameEnv + "=" + username, passwordEnv + "=" + password})
}
33 changes: 22 additions & 11 deletions cmd/npmExecuteTests_generated.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

10 changes: 7 additions & 3 deletions resources/metadata/npmExecuteTests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -57,10 +57,14 @@ spec:
- PARAMETERS
- STAGES
- STEPS
- name: credentialsEnvVarPrefix
- name: usernameEnvVar
type: string
default: "wdi5"
description: Prefix for username and password env vars.
default: "wdi5_username"
description: Env var for username.
- name: passwordEnvVar
type: string
default: "wdi5_password"
description: Env var for password.
- name: urlOptionPrefix
type: string
description: |
Expand Down

0 comments on commit d4cf486

Please sign in to comment.