yamanin/registry
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore: remove it wrappers from required variables tests (#442)

Browse Source 
## Description

<!-- Briefly describe what this PR does and why -->
Remove it wrappers from required variables tf test in jfrog-oauth and
jfrog-token modules. This solves the failing tf tests that we were
encountering in all PR's across the board.

## Type of Change

- [ ] New module
- [X] Bug fix
- [ ] Feature/enhancement
- [ ] Documentation
- [ ] Other

## Testing & Validation

- [X] Tests pass (`bun test`)
- [X] Code formatted (`bun run fmt`)
- [X] Changes tested locally

## Related Issues

<!-- Link related issues or write "None" if not applicable -->
This commit is contained in:
DevCats 2025-09-30 07:44:41 -05:00 committed by GitHub
parent e516446d03
commit 80f429faf1
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 9 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
registry/coder/modules/jfrog-oauth/main.test.ts
View File

@ -24,12 +24,10 @@ describe("jfrog-oauth", async () => {
const fakeFrogUrl = "http://localhost:8081"; const fakeFrogUrl = "http://localhost:8081";
const user = "default"; const user = "default";
it("can run apply with required variables", async () => { testRequiredVariables<TestVariables>(import.meta.dir, {
testRequiredVariables<TestVariables>(import.meta.dir, { agent_id: "some-agent-id",
agent_id: "some-agent-id", jfrog_url: fakeFrogUrl,
jfrog_url: fakeFrogUrl, package_managers: "{}",
package_managers: "{}",
});
}); });
it("generates an npmrc with scoped repos", async () => { it("generates an npmrc with scoped repos", async () => {

12
registry/coder/modules/jfrog-token/main.test.ts
View File

@ -55,13 +55,11 @@ describe("jfrog-token", async () => {
const user = "default"; const user = "default";
const token = "xxx"; const token = "xxx";
it("can run apply with required variables", async () => { testRequiredVariables<TestVariables>(import.meta.dir, {
testRequiredVariables<TestVariables>(import.meta.dir, { agent_id: "some-agent-id",
agent_id: "some-agent-id", jfrog_url: fakeFrogUrl,
jfrog_url: fakeFrogUrl, artifactory_access_token: "XXXX",
artifactory_access_token: "XXXX", package_managers: "{}",
package_managers: "{}",
});
}); });
it("generates an npmrc with scoped repos", async () => { it("generates an npmrc with scoped repos", async () => {