Skip to content

Commit

Permalink
Revert "Test that overwrite env var works in nested action (#453)" (#464
Browse files Browse the repository at this point in the history
)

This reverts commit d27529e.
  • Loading branch information
fairclothjm authored Jun 8, 2023
1 parent a87a71c commit dc4f72d
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 28 deletions.
15 changes: 0 additions & 15 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -166,21 +166,6 @@ jobs:
run: npm run test:integration:e2e
env:
OTHER_SECRET_OUTPUT: ${{ steps.kv-secrets.outputs.otherSecret }}

- name: Test Vault Action Overwrites Env Vars In Subsequent Action (part 1/2)
uses: ./
with:
url: http://localhost:8200/
token: testtoken
secrets: |
secret/data/test secret | SUBSEQUENT_TEST_SECRET;
- name: Test Vault Action Overwrites Env Vars In Subsequent Action (part 2/2)
uses: ./
with:
url: http://localhost:8200/
token: testtoken
secrets: |
secret/data/subsequent-test secret | SUBSEQUENT_TEST_SECRET;

e2e-tls:
runs-on: ubuntu-latest
Expand Down
1 change: 0 additions & 1 deletion integrationTests/e2e/e2e.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,5 @@ describe('e2e', () => {
expect(process.env.OTHERALTSECRET).toBe("OTHERCUSTOMSECRET");
expect(process.env.FOO).toBe("bar");
expect(process.env.NAMED_CUBBYSECRET).toBe("zap");
expect(process.env.SUBSEQUENT_TEST_SECRET).toBe("SUBSEQUENT_TEST_SECRET");
});
});
12 changes: 0 additions & 12 deletions integrationTests/e2e/setup.js
Original file line number Diff line number Diff line change
Expand Up @@ -76,18 +76,6 @@ const vaultToken = `${process.env.VAULT_TOKEN}` === undefined ? `${process.env.V
zip: 'zap',
},
});

await got(`http://${vaultUrl}/v1/secret/data/subsequent-test`, {
method: 'POST',
headers: {
'X-Vault-Token': vaultToken,
},
json: {
data: {
secret: 'SUBSEQUENT_TEST_SECRET',
},
},
});
} catch (error) {
console.log(error);
process.exit(1);
Expand Down

0 comments on commit dc4f72d

Please sign in to comment.