Chore: skipIfBeta is worded as the opposite logic

agent_integration_test.go

@@ -9,7 +9,7 @@ import (
 )
 
 func TestAgentsRead(t *testing.T) {
-	skipIfNotLinuxAmd64(t)
+	skipUnlessLinuxAMD64(t)
 
 	client := testClient(t)
 	ctx := context.Background()
@@ -43,7 +43,7 @@ func TestAgentsRead(t *testing.T) {
 }
 
 func TestAgentsList(t *testing.T) {
-	skipIfNotLinuxAmd64(t)
+	skipUnlessLinuxAMD64(t)
 
 	client := testClient(t)
 	ctx := context.Background()

docs/CONTRIBUTING.md

@@ -57,11 +57,11 @@ Beta features, if released, should be clearly commented:
 ExampleNewField *bool `jsonapi:"attr,example-new-field,omitempty"`
 ```
 
-When adding test cases, you can temporarily use the skipIfBeta() test helper to omit beta features from running in CI.
+When adding test cases, you can temporarily use the skipUnlessBeta() test helper to omit beta features from running in CI.
 
 ```
 t.Run("with nested changes trigger", func (t *testing.T) {
-  skipIfBeta(t)
+  skipUnlessBeta(t)
   options := WorkspaceCreateOptions {
      // rest of required fields here
      ExampleNewField: Bool(true),
@@ -70,7 +70,7 @@ t.Run("with nested changes trigger", func (t *testing.T) {
 })
 ```
 
-**Note**: After your PR has been merged, and the feature either reaches general availability, you should remove the `skipIfBeta()` flag.
+**Note**: After your PR has been merged, and the feature either reaches general availability, you should remove the `skipUnlessBeta()` flag.
 
 ## Adding New Endpoints
 

helper_test.go

@@ -2168,14 +2168,14 @@ func skipIfFreeOnly(t *testing.T) {
 }
 
 // skips a test if the test requires a beta feature
-func skipIfBeta(t *testing.T) {
+func skipUnlessBeta(t *testing.T) {
 	if !betaFeaturesEnabled() {
 		t.Skip("Skipping test related to a Terraform Cloud beta feature. Set ENABLE_BETA=1 to run.")
 	}
 }
 
 // skips a test if the architecture is not linux_amd64
-func skipIfNotLinuxAmd64(t *testing.T) {
+func skipUnlessLinuxAMD64(t *testing.T) {
 	if !linuxAmd64() {
 		t.Skip("Skipping test if architecture is not linux_amd64")
 	}

organization_integration_test.go

@@ -182,7 +182,7 @@ func TestOrganizationsRead(t *testing.T) {
 	})
 
 	t.Run("reads default project", func(t *testing.T) {
-		skipIfBeta(t)
+		skipUnlessBeta(t)
 
 		org, err := client.Organizations.ReadWithOptions(ctx, orgTest.Name, OrganizationReadOptions{Include: []OrganizationIncludeOpt{OrganizationDefaultProject}})
 		require.NoError(t, err)
@@ -520,7 +520,7 @@ func TestOrganization_Unmarshal(t *testing.T) {
 
 func TestOrganizationsReadRunTasksPermission(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()
@@ -543,7 +543,7 @@ func TestOrganizationsReadRunTasksPermission(t *testing.T) {
 func TestOrganizationsReadRunTasksEntitlement(t *testing.T) {
 	skipIfEnterprise(t)
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()

policy_evaluation_beta_test.go

@@ -10,7 +10,7 @@ import (
 
 func TestPolicyEvaluationList_Beta(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()
@@ -66,7 +66,7 @@ func TestPolicyEvaluationList_Beta(t *testing.T) {
 
 func TestPolicySetOutcomeList_Beta(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()
@@ -178,7 +178,7 @@ func TestPolicySetOutcomeList_Beta(t *testing.T) {
 
 func TestPolicySetOutcomeRead_Beta(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()

policy_integration_beta_test.go

@@ -10,7 +10,7 @@ import (
 
 func TestPoliciesCreate_Beta(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()
@@ -221,7 +221,7 @@ func TestPoliciesCreate_Beta(t *testing.T) {
 
 func TestPoliciesList_Beta(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()
@@ -309,7 +309,7 @@ func TestPoliciesList_Beta(t *testing.T) {
 
 func TestPoliciesUpdate_Beta(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()

policy_set_integration_beta_test.go

@@ -13,7 +13,7 @@ import (
 
 func TestPolicySetsList_Beta(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()
@@ -111,7 +111,7 @@ func TestPolicySetsList_Beta(t *testing.T) {
 
 func TestPolicySetsCreate_Beta(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()
@@ -334,7 +334,7 @@ func TestPolicySetsCreate_Beta(t *testing.T) {
 
 func TestPolicySetsUpdate_Beta(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()

projects_integration_test.go

@@ -10,7 +10,7 @@ import (
 )
 
 func TestProjectsList(t *testing.T) {
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()
@@ -52,7 +52,7 @@ func TestProjectsList(t *testing.T) {
 }
 
 func TestProjectsRead(t *testing.T) {
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()
@@ -84,7 +84,7 @@ func TestProjectsRead(t *testing.T) {
 }
 
 func TestProjectsCreate(t *testing.T) {
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()
@@ -136,7 +136,7 @@ func TestProjectsCreate(t *testing.T) {
 }
 
 func TestProjectsUpdate(t *testing.T) {
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()
@@ -176,7 +176,7 @@ func TestProjectsUpdate(t *testing.T) {
 }
 
 func TestProjectsDelete(t *testing.T) {
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()

registry_module_integration_test.go

@@ -142,7 +142,7 @@ func TestRegistryModulesCreate(t *testing.T) {
 		})
 
 		t.Run("with no-code attribute", func(t *testing.T) {
-			skipIfBeta(t)
+			skipUnlessBeta(t)
 			options := RegistryModuleCreateOptions{
 				Name:         String("iam"),
 				Provider:     String("aws"),
@@ -248,7 +248,7 @@ func TestRegistryModulesCreate(t *testing.T) {
 }
 
 func TestRegistryModuleUpdate(t *testing.T) {
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 	client := testClient(t)
 	ctx := context.Background()
 

task_stages_integration_beta_test.go

@@ -10,7 +10,7 @@ import (
 
 func TestTaskStagesRead_Beta(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()
@@ -108,7 +108,7 @@ func TestTaskStagesRead_Beta(t *testing.T) {
 
 func TestTaskStagesList_Beta(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()
@@ -172,7 +172,7 @@ func TestTaskStagesList_Beta(t *testing.T) {
 
 func TestTaskStageOverride_Beta(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()

team_access_integration_test.go

@@ -338,7 +338,7 @@ func TestTeamAccessesRemove(t *testing.T) {
 
 func TestTeamAccessesReadRunTasks(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 	skipIfEnterprise(t)
 
 	client := testClient(t)
@@ -368,7 +368,7 @@ func TestTeamAccessesReadRunTasks(t *testing.T) {
 
 func TestTeamAccessesUpdateRunTasks(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()

team_integration_test.go

@@ -364,7 +364,7 @@ func TestTeamCreateOptions_Marshal(t *testing.T) {
 
 func TestTeamsUpdateRunTasks(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 	skipIfEnterprise(t)
 
 	client := testClient(t)

workspace_integration_test.go

@@ -353,7 +353,7 @@ func TestWorkspacesCreate(t *testing.T) {
 	t.Cleanup(orgTestCleanup)
 
 	t.Run("with valid project option", func(t *testing.T) {
-		skipIfBeta(t)
+		skipUnlessBeta(t)
 
 		options := WorkspaceCreateOptions{
 			Name:                       String(fmt.Sprintf("foo-%s", randomString(t))),
@@ -824,7 +824,7 @@ func TestWorkspacesUpdate(t *testing.T) {
 	})
 
 	t.Run("when updating project", func(t *testing.T) {
-		skipIfBeta(t)
+		skipUnlessBeta(t)
 
 		kBefore, kTestCleanup := createProject(t, client, orgTest)
 		defer kTestCleanup()
@@ -2129,7 +2129,7 @@ func TestWorkspaceCreateOptions_Marshal(t *testing.T) {
 
 func TestWorkspacesRunTasksPermission(t *testing.T) {
 	skipIfFreeOnly(t)
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 
 	client := testClient(t)
 	ctx := context.Background()
@@ -2149,7 +2149,7 @@ func TestWorkspacesRunTasksPermission(t *testing.T) {
 }
 
 func TestWorkspacesProjects(t *testing.T) {
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 	client := testClient(t)
 	ctx := context.Background()
 

workspace_run_task_integration_test.go

@@ -46,7 +46,7 @@ func TestWorkspaceRunTasksCreate(t *testing.T) {
 
 func TestWorkspaceRunTasksCreateBeta(t *testing.T) {
 	// Once Pre-Plan Tasks are generally available, this can replace the above TestWorkspaceRunTasksCreate
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 	skipIfFreeOnly(t)
 
 	client := testClient(t)
@@ -188,7 +188,7 @@ func TestWorkspaceRunTasksUpdate(t *testing.T) {
 
 func TestWorkspaceRunTasksUpdateBeta(t *testing.T) {
 	// Once Pre-Plan Tasks are generally available, this can replace the above TestWorkspaceRunTasksUpdate
-	skipIfBeta(t)
+	skipUnlessBeta(t)
 	skipIfFreeOnly(t)
 
 	client := testClient(t)