Fix tests

The provider template is deprecated. This provider has been used extensively in tests (both in the upstream Terragrunt and within our fork) because `data.template_file` was seen as a simple Terraform resource that didn’t produce any real artifacts. However, while the Linux versions are still available for download, the Windows and OSX/Darwin versions are no longer offered, causing test failures in local setups on Mac or Windows. This makes it challenging to add or modify tests locally.

.github/workflows/pr.yml

@@ -42,9 +42,9 @@ jobs:
     strategy:
       matrix:
         terraform:
-          - '1.0.2'
-          - '1.3.8'
-          - '1.4.0'
+          - '1.7.5'
+          - '1.8.5'
+          - '1.9.7'
     steps:
       - uses: actions/setup-go@v5
         with:

configstack/module_test.go

@@ -46,7 +46,7 @@ func TestResolveTerraformModulesOneModuleWithIncludesNoDependencies(t *testing.T
 		Path:         canonical(t, "../test/fixture-modules/module-b/module-b-child"),
 		Dependencies: []*TerraformModule{},
 		Config: config.TerragruntConfig{
-			RemoteState: state(t, "bucket", "module-b-child/terraform.tfstate"),
+			RemoteState: state("bucket", "module-b-child/terraform.tfstate"),
 			Terraform:   &config.TerraformConfig{Source: "..."},
 		},
 		TerragruntOptions: mockOptions.Clone(canonical(t, "../test/fixture-modules/module-b/module-b-child/"+config.DefaultConfigName)),
@@ -104,7 +104,7 @@ func TestResolveTerraformModulesMultipleModulesWithDependencies(t *testing.T) {
 		Path:         canonical(t, "../test/fixture-modules/module-b/module-b-child"),
 		Dependencies: []*TerraformModule{},
 		Config: config.TerragruntConfig{
-			RemoteState: state(t, "bucket", "module-b-child/terraform.tfstate"),
+			RemoteState: state("bucket", "module-b-child/terraform.tfstate"),
 			Terraform:   &config.TerraformConfig{Source: "..."},
 		},
 		TerragruntOptions: mockOptions.Clone(canonical(t, "../test/fixture-modules/module-b/module-b-child/"+config.DefaultConfigName)),
@@ -152,7 +152,7 @@ func TestResolveTerraformModulesMultipleModulesWithDependenciesWithIncludes(t *t
 		Path:         canonical(t, "../test/fixture-modules/module-b/module-b-child"),
 		Dependencies: []*TerraformModule{},
 		Config: config.TerragruntConfig{
-			RemoteState: state(t, "bucket", "module-b-child/terraform.tfstate"),
+			RemoteState: state("bucket", "module-b-child/terraform.tfstate"),
 			Terraform:   &config.TerraformConfig{Source: "..."},
 		},
 		TerragruntOptions: mockOptions.Clone(canonical(t, "../test/fixture-modules/module-b/module-b-child/"+config.DefaultConfigName)),
@@ -162,7 +162,7 @@ func TestResolveTerraformModulesMultipleModulesWithDependenciesWithIncludes(t *t
 		Path:         canonical(t, "../test/fixture-modules/module-e/module-e-child"),
 		Dependencies: []*TerraformModule{moduleA, moduleB},
 		Config: config.TerragruntConfig{
-			RemoteState:  state(t, "bucket", "module-e-child/terraform.tfstate"),
+			RemoteState:  state("bucket", "module-e-child/terraform.tfstate"),
 			Dependencies: &config.ModuleDependencies{Paths: []string{Abs("../test/fixture-modules/module-a"), Abs("../test/fixture-modules/module-b/module-b-child")}},
 			Terraform:    &config.TerraformConfig{Source: "test"},
 		},

configstack/stack_test.go

@@ -17,14 +17,14 @@ func TestFindStackInSubfolders(t *testing.T) {
 	t.Parallel()
 
 	filePaths := []string{
-		"/stage/data-stores/redis/" + config.DefaultConfigName,
-		"/stage/data-stores/postgres/" + config.DefaultConfigName,
-		"/stage/ecs-cluster/" + config.DefaultConfigName,
-		"/stage/kms-master-key/" + config.DefaultConfigName,
-		"/stage/vpc/" + config.DefaultConfigName,
+		"stage/data-stores/redis/" + config.DefaultConfigName,
+		"stage/data-stores/postgres/" + config.DefaultConfigName,
+		"stage/ecs-cluster/" + config.DefaultConfigName,
+		"stage/kms-master-key/" + config.DefaultConfigName,
+		"stage/vpc/" + config.DefaultConfigName,
 	}
 
-	tempFolder := createTempFolder(t)
+	tempFolder := createTempFolder()
 	writeDummyTerragruntConfigs(t, tempFolder, filePaths)
 
 	envFolder := filepath.ToSlash(util.JoinPath(tempFolder + "/stage"))
@@ -51,13 +51,8 @@ func TestFindStackInSubfolders(t *testing.T) {
 	}
 }
 
-func createTempFolder(t *testing.T) string {
-	tmpFolder, err := ioutil.TempDir("", "")
-	if err != nil {
-		t.Fatalf("Failed to create temp directory: %s\n", err.Error())
-	}
-
-	return filepath.ToSlash(tmpFolder)
+func createTempFolder() string {
+	return filepath.ToSlash(os.TempDir())
 }
 
 // Create a dummy Terragrunt config file at each of the given paths

configstack/test_helpers.go

@@ -111,7 +111,7 @@ func assertRunningModulesEqual(t *testing.T, expected *runningModule, actual *ru
 }
 
 // We can't do a simple IsError comparison for UnrecognizedDependency because that error is a struct that
-// contains an array, and in Go, trying to compare arrays gives a "comparing noncomparable type
+// contains an array, and in Go, trying to compare arrays gives a "comparing non comparable type
 // configstack.UnrecognizedDependency" panic. Therefore, we have to compare that error more manually.
 func assertErrorsEqual(t *testing.T, expected error, actual error, messageAndArgs ...interface{}) {
 	actual = tgerrors.Unwrap(actual)
@@ -165,7 +165,7 @@ func canonical(t *testing.T, path string) string {
 }
 
 // Create a State struct
-func state(t *testing.T, bucket string, key string) *remote.State {
+func state(bucket string, key string) *remote.State {
 	return &remote.State{
 		Backend: "s3",
 		Config: map[string]interface{}{

test/fixture-download/hello-world-with-backend/main.tf

@@ -1,13 +1,9 @@
-data "template_file" "test" {
-  template = "hello, ${var.name}"
-}
-
 variable "name" {
   description = "Specify a name"
 }
 
-output "test" {
-  value = data.template_file.test.rendered
+output "example" {
+  value = "hello, ${var.name}"
 }
 
 terraform {

test/fixture-download/hello-world/main.tf

@@ -1,13 +1,9 @@
-data "template_file" "test" {
-  template = "${module.hello.hello}, ${var.name}"
-}
-
 variable "name" {
   description = "Specify a name"
 }
 
 output "test" {
-  value = data.template_file.test.rendered
+  value = "${module.hello.hello}, ${var.name}"
 }
 
 module "hello" {

test/fixture-download/local-relative-extra-args-windows/JZwoL6Viko8bzuRvTOQFx3Jh8vs/3mU4huxMLOXOW5ZgJOFXGUFDKc8/main.tf

@@ -1,13 +1,9 @@
-data "template_file" "test" {
-  template = "${module.hello.hello}, ${var.name}"
-}
-
 variable "name" {
   description = "Specify a name"
 }
 
 output "test" {
-  value = data.template_file.test.rendered
+  value = "${module.hello.hello}, ${var.name}"
 }
 
 module "hello" {

test/fixture-gotemplate/main.tf

@@ -1,9 +1,5 @@
-data "template_file" "@String(`This is an example`).Fields().Join(`_`)" {
-  template = "ok"
-}
-
-output "result" {
-  value = data.template_file.This_is_an_example.rendered
+output "@String(`This is an output`).Fields().Join(`_`)" {
+  value = "ok"
 }
 
 variable "test1" {

test/fixture-hooks/after-only/main.tf

@@ -1,7 +1,3 @@
-data "template_file" "example" {
-  template = "hello, world"
-}
-
 output "example" {
-  value = data.template_file.example.rendered
+  value = "hello, world"
 }

test/fixture-hooks/before-and-after-merge/qa/my-app/main.tf

@@ -2,10 +2,6 @@ terraform {
   backend "s3" {}
 }
 
-data "template_file" "example" {
-  template = "hello, world"
-}
-
 output "example" {
-  value = data.template_file.example.rendered
+  value = "hello, world"
 }

test/fixture-hooks/before-and-after/main.tf

@@ -1,7 +1,3 @@
-data "template_file" "example" {
-  template = "hello, world"
-}
-
 output "example" {
-  value = data.template_file.example.rendered
+  value = "hello, world"
 }

test/fixture-hooks/before-only/main.tf

@@ -1,7 +1,3 @@
-data "template_file" "example" {
-  template = "hello, world"
-}
-
 output "example" {
-  value = data.template_file.example.rendered
+  value = "hello, world"
 }

test/fixture-hooks/before-only/terragrunt.hcl

@@ -1,4 +1,4 @@
 pre_hook "pre_hook_1" {
   on_commands = ["apply", "plan"]
   command     = "touch file.out"
-}
+}

test/fixture-hooks/exitcode-1-run-on-errors/main.tf

@@ -1,7 +1,3 @@
-data "template_file" "example" {
-  template = "hello, world"
-}
-
 output "example" {
-  value = data.template_file.example.rendered
+  value = "hello, world"
 }

test/fixture-hooks/exitcode-1/main.tf

@@ -1,7 +1,3 @@
-data "template_file" "example" {
-  template = "hello, world"
-}
-
 output "example" {
-  value = data.template_file.example.rendered
+  value = "hello, world"
 }

test/fixture-hooks/exitcode-1/terragrunt.hcl

@@ -6,4 +6,4 @@ post_hook "post_hook_1" {
 post_hook "post_hook_2" {
   on_commands = ["apply", "plan"]
   command     = "touch test.out"
-}
+}

test/fixture-hooks/exitcode-2-pre/main.tf

@@ -1,7 +1,3 @@
-data "template_file" "example" {
-  template = "hello, world"
-}
-
 output "example" {
-  value = data.template_file.example.rendered
+  value = "hello, world"
 }

test/fixture-hooks/exitcode-2/main.tf

@@ -1,7 +1,3 @@
-data "template_file" "example" {
-  template = "hello, world"
-}
-
 output "example" {
-  value = data.template_file.example.rendered
+  value = "hello, world"
 }

test/fixture-hooks/ignore-error/main.tf

@@ -1,7 +1,3 @@
-data "template_file" "example" {
-  template = "hello, world"
-}
-
 output "example" {
-  value = data.template_file.example.rendered
+  value = "hello, world"
 }

test/fixture-hooks/interpolations/main.tf

@@ -1,7 +1,3 @@
-data "template_file" "example" {
-  template = "hello, world"
-}
-
 output "example" {
-  value = data.template_file.example.rendered
+  value = "hello, world"
 }

test/fixture-import-files/bad-source/main.tf

@@ -1,7 +1,7 @@
-data "template_file" "example" {
-  template = var.var
+output "example" {
+  value = var.var
 }
 
-output "example" {
-  value = data.template_file.example.rendered
+variable "var" {
+  default = "123"
 }

test/fixture-import-files/basic/main.tf

@@ -1,7 +1,3 @@
-data "template_file" "example" {
-  template = var.var
-}
-
 output "example" {
-  value = data.template_file.example.rendered
+  value = var.var
 }

test/fixture-import-files/overwrite/main.tf

@@ -1,7 +1,3 @@
-data "template_file" "example" {
-  template = var.var
-}
-
 output "example" {
-  value = data.template_file.example.rendered
+  value = var.var
 }

test/fixture-include/qa/my-app/main.tf

@@ -2,7 +2,6 @@ terraform {
   backend "s3" {}
 }
 
-# Create an arbitrary local resource
-data "template_file" "test" {
-  template = "Hello, I am a template."
-}
+output "text" {
+  value = "Hello, I am an output"
+}

test/fixture-parent-folders/terragrunt-in-root/child/sub-child/sub-sub-child/main.tf

@@ -2,7 +2,6 @@ terraform {
   backend "s3" {}
 }
 
-# Create an arbitrary local resource
-data "template_file" "test" {
-  template = "Hello, I am a template."
-}
+output "test" {
+  value = "Hello, I am a output"
+}

test/fixture-stack/mgmt/bastion-host/main.tf

@@ -2,13 +2,8 @@ terraform {
   backend "s3" {}
 }
 
-# Create an arbitrary local resource
-data "template_file" "text" {
-  template = "[I am a bastion-host template. Data from my dependencies: vpc = ${data.terraform_remote_state.vpc.outputs.text}]"
-}
-
 output "text" {
-  value = data.template_file.text.rendered
+  value = "[I am a bastion-host template. Data from my dependencies: vpc = ${data.terraform_remote_state.vpc.outputs.text}]"
 }
 
 variable "terraform_remote_state_s3_bucket" {

test/fixture-stack/mgmt/vpc/main.tf

@@ -6,11 +6,6 @@ terraform {
   backend "s3" {}
 }
 
-# Create an arbitrary local resource
-data "template_file" "text" {
-  template = "[I am a mgmt vpc template. I have no dependencies.]"
-}
-
 output "text" {
-  value = data.template_file.text.rendered
+  value = "[I am a mgmt vpc template. I have no dependencies.]"
 }

test/fixture-stack/stage/backend-app/main.tf

@@ -2,13 +2,8 @@ terraform {
   backend "s3" {}
 }
 
-# Create an arbitrary local resource
-data "template_file" "text" {
-  template = "[I am a backend-app template. Data from my dependencies: vpc = ${data.terraform_remote_state.vpc.outputs.text}, bastion-host = ${data.terraform_remote_state.bastion_host.outputs.text}, mysql = ${data.terraform_remote_state.mysql.outputs.text}, search-app = ${data.terraform_remote_state.search_app.outputs.text}]"
-}
-
 output "text" {
-  value = data.template_file.text.rendered
+  value = "[I am a backend-app template. Data from my dependencies: vpc = ${data.terraform_remote_state.vpc.outputs.text}, bastion-host = ${data.terraform_remote_state.bastion_host.outputs.text}, mysql = ${data.terraform_remote_state.mysql.outputs.text}, search-app = ${data.terraform_remote_state.search_app.outputs.text}]"
 }
 
 variable "terraform_remote_state_s3_bucket" {