Skip to content

Commit 80f429f

Browse files
chore: remove it wrappers from required variables tests (#442)
## 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 -->
1 parent e516446 commit 80f429f

File tree

2 files changed

+9
-13
lines changed

2 files changed

+9
-13
lines changed

registry/coder/modules/jfrog-oauth/main.test.ts

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,12 +24,10 @@ describe("jfrog-oauth", async () => {
2424
const fakeFrogUrl = "http://localhost:8081";
2525
const user = "default";
2626

27-
it("can run apply with required variables", async () => {
28-
testRequiredVariables<TestVariables>(import.meta.dir, {
29-
agent_id: "some-agent-id",
30-
jfrog_url: fakeFrogUrl,
31-
package_managers: "{}",
32-
});
27+
testRequiredVariables<TestVariables>(import.meta.dir, {
28+
agent_id: "some-agent-id",
29+
jfrog_url: fakeFrogUrl,
30+
package_managers: "{}",
3331
});
3432

3533
it("generates an npmrc with scoped repos", async () => {

registry/coder/modules/jfrog-token/main.test.ts

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -55,13 +55,11 @@ describe("jfrog-token", async () => {
5555
const user = "default";
5656
const token = "xxx";
5757

58-
it("can run apply with required variables", async () => {
59-
testRequiredVariables<TestVariables>(import.meta.dir, {
60-
agent_id: "some-agent-id",
61-
jfrog_url: fakeFrogUrl,
62-
artifactory_access_token: "XXXX",
63-
package_managers: "{}",
64-
});
58+
testRequiredVariables<TestVariables>(import.meta.dir, {
59+
agent_id: "some-agent-id",
60+
jfrog_url: fakeFrogUrl,
61+
artifactory_access_token: "XXXX",
62+
package_managers: "{}",
6563
});
6664

6765
it("generates an npmrc with scoped repos", async () => {

0 commit comments

Comments
 (0)