Refactor test helper and remove unused context

Renamed testAccTFERegistryModule_vcsInvalidBranch to testAccTFERegistryModule_invalidTestConfig for clarity and updated its usage. Also removed the unused context variable from the test file.

Author: hashimoon

internal/provider/resource_tfe_registry_module_test.go

@@ -4,7 +4,6 @@
 package provider
 
 import (
-	"context"
 	"fmt"
 	"math/rand"
 	"regexp"
@@ -18,8 +17,6 @@ import (
 	"github.com/hashicorp/terraform-plugin-testing/terraform"
 )
 
-var ctx = context.Background()
-
 func TestAccTFERegistryModule_vcsBasic(t *testing.T) {
 	registryModule := &tfe.RegistryModule{}
 	rInt := rand.New(rand.NewSource(time.Now().UnixNano())).Int()
@@ -716,7 +713,7 @@ func TestAccTFERegistryModule_invalidTestConfigOnUpdate(t *testing.T) {
 				),
 			},
 			{
-				Config:      testAccTFERegistryModule_vcsInvalidBranch(rInt),
+				Config:      testAccTFERegistryModule_invalidTestConfig(rInt),
 				ExpectError: regexp.MustCompile(`tests_enabled must be provided when configuring a test_config`),
 			},
 		},
@@ -1216,7 +1213,7 @@ resource "tfe_registry_module" "foobar" {
 		envGithubRegistryModuleIdentifer,
 		envGithubRegistryModuleIdentifer)
 }
-func testAccTFERegistryModule_vcsInvalidBranch(rInt int) string {
+func testAccTFERegistryModule_invalidTestConfig(rInt int) string {
 	return fmt.Sprintf(`
 resource "tfe_organization" "foobar" {
  name  = "tst-terraform-%d"