[TF-7737] Ensure TestsEnabled is present before validating

hashimoon · hashimoon · commit 3ef709824615 · 2023-10-03T16:35:04.000-07:00
diff --git a/registry_module.go b/registry_module.go
@@ -727,9 +727,11 @@ func (o RegistryModuleCreateWithVCSConnectionOptions) valid() error {
 	}
 
 	if o.TestConfig != nil {
-		if *o.TestConfig.TestsEnabled {
-			if !validString(o.VCSRepo.Branch) {
-				return ErrRequiredBranchWhenTestsEnabled
+		if o.TestConfig.TestsEnabled != nil {
+			if *o.TestConfig.TestsEnabled {
+				if !validString(o.VCSRepo.Branch) {
+					return ErrRequiredBranchWhenTestsEnabled
+				}
 			}
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -727,9 +727,11 @@ func (o RegistryModuleCreateWithVCSConnectionOptions) valid() error {`
`727`	`727`	`}`
`728`	`728`
`729`	`729`	`if o.TestConfig != nil {`
`730`		`- if *o.TestConfig.TestsEnabled {`
`731`		`- if !validString(o.VCSRepo.Branch) {`
`732`		`- return ErrRequiredBranchWhenTestsEnabled`
	`730`	`+ if o.TestConfig.TestsEnabled != nil {`
	`731`	`+ if *o.TestConfig.TestsEnabled {`
	`732`	`+ if !validString(o.VCSRepo.Branch) {`
	`733`	`+ return ErrRequiredBranchWhenTestsEnabled`
	`734`	`+ }`
`733`	`735`	`}`
`734`	`736`	`}`
`735`	`737`	`}`