update comment

Liam Cervante · Liam Cervante · commit 61d1d0e86edc · 2025-06-03T09:41:05.000+02:00
diff --git a/internal/moduletest/graph/variables.go b/internal/moduletest/graph/variables.go
@@ -150,9 +150,10 @@ func (n *NodeTestRun) GetVariables(ctx *EvalContext, includeWarnings bool) (terr
 			}
 		}
 
-		// Finally, we're going to give these variables a value. They'll be
-		// processed by the Terraform graph and provided a default value later
-		// if they have one.
+		// If all else fails, these variables may have default values set within
+		// the to-be-executed Terraform config. We'll put in placeholder values
+		// if that is the case, otherwise add a diagnostic early to avoid
+		// executing something we know will fail.
 
 		if variable.Required() {
 			diags = diags.Append(&hcl.Diagnostic{

Original file line number	Diff line number	Diff line change
`@@ -150,9 +150,10 @@ func (n NodeTestRun) GetVariables(ctx EvalContext, includeWarnings bool) (terr`
`150`	`150`	`}`
`151`	`151`	`}`
`152`	`152`
`153`		`- // Finally, we're going to give these variables a value. They'll be`
`154`		`- // processed by the Terraform graph and provided a default value later`
`155`		`- // if they have one.`
	`153`	`+ // If all else fails, these variables may have default values set within`
	`154`	`+ // the to-be-executed Terraform config. We'll put in placeholder values`
	`155`	`+ // if that is the case, otherwise add a diagnostic early to avoid`
	`156`	`+ // executing something we know will fail.`
`156`	`157`
`157`	`158`	`if variable.Required() {`
`158`	`159`	`diags = diags.Append(&hcl.Diagnostic{`