fix recursive variable reference resolution

pkg/iac-providers/terraform/commons/variable-references.go

@@ -148,9 +148,9 @@ func (r *RefResolver) ResolveVarRefFromParentModuleCall(varRef string) interface
 	if reflect.TypeOf(val).Kind() == reflect.String {
 		valStr := val.(string)
 		resolvedVal := strings.Replace(varRef, varExpr, valStr, 1)
-		if varRef == resolvedVal {
+		if strings.Contains(valStr, varExpr) {
 			zap.S().Debugf("resolved str variable ref refers to self: '%v'", varRef)
-			return varRef
+			return resolvedVal
 		}
 		zap.S().Debugf("resolved str variable ref: '%v', value: '%v'", varRef, string(resolvedVal))
 		return r.ResolveStrRef(resolvedVal)

pkg/iac-providers/terraform/v14/load-dir_test.go

@@ -119,6 +119,13 @@ func TestLoadIacDir(t *testing.T) {
 			tfv14:       TfV14{},
 			wantErr:     nil,
 		},
+		{
+			name:        "recursive loop while resolving variables",
+			tfConfigDir: filepath.Join(testDataDir, "recursive-loop"),
+			tfJSONFile:  filepath.Join(tfJSONDir, "recursive-loop.json"),
+			tfv14:       TfV14{},
+			wantErr:     nil,
+		},
 	}
 
 	for _, tt := range table2 {

pkg/iac-providers/terraform/v14/testdata/recursive-loop/bug.tf

@@ -0,0 +1,9 @@
+variable "filename" {
+  type = string
+}
+
+module "dummy" {
+  source = "./dummy"
+
+  filename = "${path.module}/${var.filename}"
+}

pkg/iac-providers/terraform/v14/testdata/recursive-loop/dummy/main.tf

@@ -0,0 +1,12 @@
+variable "filename" {
+  type = string
+}
+
+resource "null_resource" "example" {
+  container_definitions = templatefile(
+    var.filename,
+    {
+      foo = "bar"
+    }
+  )
+}

pkg/iac-providers/terraform/v14/testdata/tfjson/recursive-loop.json

@@ -0,0 +1,15 @@
+{
+    "null_resource": [
+      {
+        "id": "null_resource.example",
+        "name": "example",
+        "source": "dummy/main.tf",
+        "line": 5,
+        "type": "null_resource",
+        "config": {
+          "container_definitions": "${templatefile(\n    ${path.module}/${var.filename},\n    {\n      foo = \"bar\"\n    }\n  )}"
+        },
+        "skip_rules": null
+      }
+    ]
+  }