test: fix windows CI tests.

Signed-off-by: i4k <t.nateldemoura@gmail.com>

Author: i4ki

.github/workflows/ci-experimental.yml .github/workflows/ci-windows.yml.github/workflows/ci-experimental.yml renamed to .github/workflows/ci-windows.yml

@@ -1,15 +1,9 @@
 # Copyright 2023 Terramate GmbH
 # SPDX-License-Identifier: MPL-2.0
 
-name: ci-experimental
+name: ci-windows
 on:
   push:
-    paths-ignore:
-      - 'docs/**'
-      - 'makefiles/**'
-      - '.github/**'
-      - '**/*.tm.hcl'
-      - '.tool-versions'
 
 jobs:
   build_test:

e2etests/cloud/run_script_cloud_preview_test.go

@@ -5,7 +5,6 @@ package cloud_test
 
 import (
 	"encoding/json"
-	"fmt"
 	"net/http"
 	"os"
 	"path/filepath"
@@ -72,22 +71,20 @@ func TestScriptRunWithCloudSyncPreview(t *testing.T) {
 					}
 				  }
 				`,
-				fmt.Sprintf(`f:stack/preview.tm:
+				`f:stack/preview.tm:
 
 				  script "preview" {
 					description = "sync a preview"
 					job {
 					  commands = [
-						["%s", "plan", "-out=out.tfplan", "-no-color", "-detailed-exitcode", {
+						["terraform", "plan", "-out=out.tfplan", "-no-color", "-detailed-exitcode", {
 						  sync_preview             = true,
 						  terraform_plan_file = "out.tfplan",
 						}],
 					  ]
 					}
 				  }
 				  `,
-					TerraformTestPath,
-				),
 				`f:stack/main.tf:
 				  resource "local_file" "foo" {
 					content  = "test content"
@@ -129,22 +126,20 @@ func TestScriptRunWithCloudSyncPreview(t *testing.T) {
 					}
 				  }
 				`,
-				fmt.Sprintf(`f:stack/preview.tm:
+				`f:stack/preview.tm:
 
 				  script "preview" {
 					description = "sync a preview"
 					job {
 					  commands = [
-						["%s", "plan", "-out=out.tfplan", "-no-color", "-detailed-exitcode", {
+						["terraform", "plan", "-out=out.tfplan", "-no-color", "-detailed-exitcode", {
 						  sync_preview             = true,
 						  terraform_plan_file = "out.tfplan",
 						}],
 					  ]
 					}
 				  }
 				  `,
-					TerraformTestPath,
-				),
 				`f:stack/main.tf:
 				  resource "local_file" "foo" {
 					content  = "test content"
@@ -177,7 +172,7 @@ func TestScriptRunWithCloudSyncPreview(t *testing.T) {
 						{
 							ID:     "1",
 							Status: "changed",
-							Cmd:    []string{TerraformTestPath, "plan", "-out=out.tfplan", "-no-color", "-detailed-exitcode"},
+							Cmd:    []string{"terraform", "plan", "-out=out.tfplan", "-no-color", "-detailed-exitcode"},
 						},
 					},
 					ReviewRequest: &cloud.ReviewRequest{
@@ -239,23 +234,21 @@ func TestScriptRunWithCloudSyncPreview(t *testing.T) {
 					}
 				  }
 				`,
-				fmt.Sprintf(`f:stack/preview.tm:
+				`f:stack/preview.tm:
 
 				  script "preview" {
 					description = "sync a preview"
 					job {
 					  commands = [
 					    ["do-not-exist-command"],
-						["%s", "plan", "-out=out.tfplan", "-no-color", "-detailed-exitcode", {
+						["terraform", "plan", "-out=out.tfplan", "-no-color", "-detailed-exitcode", {
 						  sync_preview             = true,
 						  terraform_plan_file = "out.tfplan",
 						}],
 					  ]
 					}
 				  }
 				  `,
-					TerraformTestPath,
-				),
 				`f:stack/main.tf:
 				  resource "local_file" "foo" {
 					content  = "test content"
@@ -285,7 +278,7 @@ func TestScriptRunWithCloudSyncPreview(t *testing.T) {
 						{
 							ID:     "1",
 							Status: "failed",
-							Cmd:    []string{TerraformTestPath, "plan", "-out=out.tfplan", "-no-color", "-detailed-exitcode"},
+							Cmd:    []string{"terraform", "plan", "-out=out.tfplan", "-no-color", "-detailed-exitcode"},
 						},
 					},
 					ReviewRequest: &cloud.ReviewRequest{

e2etests/core/run_test.go

@@ -2613,8 +2613,8 @@ func TestRunOutput(t *testing.T) {
 			name:    "run without eval",
 			runArgs: []string{HelperPath, "echo", "hello"},
 			want: RunExpected{
-				Stderr: "terramate: Entering stack in /stack" + "\n" +
-					fmt.Sprintf(`terramate: Executing command "%s echo hello"`, HelperPath) + "\n",
+				StderrRegex: "terramate: Entering stack in /stack" + "\n" +
+					`terramate: Executing command ".*? echo hello"` + "\n",
 				Stdout: "hello\n",
 			},
 		},

e2etests/internal/runner/setup.go

@@ -23,6 +23,10 @@ var TerraformVersion string
 // TerraformTestPath is the path to the installed terraform binary.
 var TerraformTestPath string
 
+// TerraformTestPathAsHCL is the path to the installed terraform binary
+// but suited for including in an HCL string expression.
+var TerraformTestPathAsHCL string
+
 var terraformCleanup func()
 
 // HelperPath is the path to the test binary we compiled for test purposes
@@ -64,6 +68,10 @@ func Setup(projectRoot string) (err error) {
 			err = errors.E(err, "failed to setup Terraform binary")
 			return
 		}
+		TerraformTestPathAsHCL = fmt.Sprintf(`${tm_chomp(<<-EOF
+		%s
+	EOF
+	)}`, TerraformTestPath)
 	})
 
 	if err == nil {

makefiles/windows.mk

@@ -34,8 +34,10 @@ tempdir=$(shell .\bin\helper.exe tempdir)
 test: test/helper build
 	set TM_TEST_ROOT_TEMPDIR=$(tempdir)
 	go test -timeout 30m -p 100 ./...
+	set status=%errorlevel%
 	.\bin\helper.exe rm $(tempdir)
 	.\bin\terramate.exe run -- helper.exe true
+	exit %status%
 
  ## remove build artifacts
 .PHONY: clean