From 80f429faf10a4e31ba3ab8849c439e96e221c7d0 Mon Sep 17 00:00:00 2001
From: DevCats <christofer@coder.com>
Date: Tue, 30 Sep 2025 07:44:41 -0500
Subject: [PATCH] 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 -->
---
 registry/coder/modules/jfrog-oauth/main.test.ts | 10 ++++------
 registry/coder/modules/jfrog-token/main.test.ts | 12 +++++-------
 2 files changed, 9 insertions(+), 13 deletions(-)

diff --git a/registry/coder/modules/jfrog-oauth/main.test.ts b/registry/coder/modules/jfrog-oauth/main.test.ts
index 0a8e62c0..940d166b 100644
--- a/registry/coder/modules/jfrog-oauth/main.test.ts
+++ b/registry/coder/modules/jfrog-oauth/main.test.ts
@@ -24,12 +24,10 @@ describe("jfrog-oauth", async () => {
   const fakeFrogUrl = "http://localhost:8081";
   const user = "default";
 
-  it("can run apply with required variables", async () => {
-    testRequiredVariables<TestVariables>(import.meta.dir, {
-      agent_id: "some-agent-id",
-      jfrog_url: fakeFrogUrl,
-      package_managers: "{}",
-    });
+  testRequiredVariables<TestVariables>(import.meta.dir, {
+    agent_id: "some-agent-id",
+    jfrog_url: fakeFrogUrl,
+    package_managers: "{}",
   });
 
   it("generates an npmrc with scoped repos", async () => {
diff --git a/registry/coder/modules/jfrog-token/main.test.ts b/registry/coder/modules/jfrog-token/main.test.ts
index 9e3097b0..419b5f28 100644
--- a/registry/coder/modules/jfrog-token/main.test.ts
+++ b/registry/coder/modules/jfrog-token/main.test.ts
@@ -55,13 +55,11 @@ describe("jfrog-token", async () => {
   const user = "default";
   const token = "xxx";
 
-  it("can run apply with required variables", async () => {
-    testRequiredVariables<TestVariables>(import.meta.dir, {
-      agent_id: "some-agent-id",
-      jfrog_url: fakeFrogUrl,
-      artifactory_access_token: "XXXX",
-      package_managers: "{}",
-    });
+  testRequiredVariables<TestVariables>(import.meta.dir, {
+    agent_id: "some-agent-id",
+    jfrog_url: fakeFrogUrl,
+    artifactory_access_token: "XXXX",
+    package_managers: "{}",
   });
 
   it("generates an npmrc with scoped repos", async () => {