fix: test for github actions (#0)

Signed-off-by: tkrop <[email protected]>

.github/workflows/go.yaml

@@ -15,7 +15,8 @@ jobs:
     - name: Build and tests
       env:
         CODACY_PROJECT_TOKEN: ${{ secrets.CODACY_PROJECT_TOKEN }}
-      run: make --file=Makefile.base --trace all
+        LANG: en_US.UTF-8
+      run: make --file=Makefile.base all
 
     - name: Send coverage report
       uses: shogo82148/actions-goveralls@v1

Makefile.base

@@ -410,7 +410,7 @@ bump:
 	    patsplit($$0, v, "[0-9]*", seps); len = length(seps); \
 	    if (arg !~ "-$$") {v[pos]++ } else { v[pos]-- }; \
 	    for (i = 1; i < len; i++) { printf("%d%s", v[i], seps[i]) } \
-	  }' VERSION)"; echo $${VERSION}; \
+	  }' VERSION)"; \
 	elif ! [[ "$(RUNARGS)" =~ ^[0-9]+(\.[0-9]+){0,2}(-.*)?$$ ]]; then \
 	  echo "error: invalid new version [$(RUNARGS)]"; exit 1; \
 	else VERSION="$(RUNARGS)"; fi; echo $${VERSION} >VERSION; \

VERSION

@@ -1 +1 @@
-0.0.6
+0.0.5

internal/make/fixtures/bash-trace.err

@@ -1,2 +1,2 @@
 call: --trace --completion=bash
-info: {"path":"github.com/tkrop/go-make","repo":"[email protected]:tkrop/go-make","build":"0001-01-01T00:00:00Z","commit":"0001-01-01T00:00:00Z","dirty":true,"go":"1.21.4","platform":"linux/amd64","compiler":"gc"}
+info: {"path":"github.com/tkrop/go-make","repo":"[email protected]:tkrop/go-make","build":"0001-01-01T00:00:00Z","commit":"0001-01-01T00:00:00Z","dirty":true,"go":"{{GOVERSION}}","platform":"{{PLATFORM}}","compiler":"{{COMPILER}}"}

internal/make/fixtures/targets-trace.err

@@ -1,4 +1,4 @@
 call: targets --trace
-info: {"path":"github.com/tkrop/go-make","repo":"[email protected]:tkrop/go-make","build":"0001-01-01T00:00:00Z","commit":"0001-01-01T00:00:00Z","dirty":true,"go":"1.21.4","platform":"linux/amd64","compiler":"gc"}
+info: {"path":"github.com/tkrop/go-make","repo":"[email protected]:tkrop/go-make","build":"0001-01-01T00:00:00Z","commit":"0001-01-01T00:00:00Z","dirty":true,"go":"{{GOVERSION}}","platform":"{{PLATFORM}}","compiler":"{{COMPILER}}"}
 exec: make --file Makefile.base targets --trace [go-make/internal/make]
 Makefile.base:18: info: please customize variables in Makefile.vars

internal/make/fixtures/version-trace.err

@@ -1,2 +1,2 @@
 call: --trace --version
-info: {"path":"github.com/tkrop/go-make","repo":"[email protected]:tkrop/go-make","build":"0001-01-01T00:00:00Z","commit":"0001-01-01T00:00:00Z","dirty":true,"go":"1.21.4","platform":"linux/amd64","compiler":"gc"}
+info: {"path":"github.com/tkrop/go-make","repo":"[email protected]:tkrop/go-make","build":"0001-01-01T00:00:00Z","commit":"0001-01-01T00:00:00Z","dirty":true,"go":"{{GOVERSION}}","platform":"{{PLATFORM}}","compiler":"{{COMPILER}}"}

internal/make/fixtures/version.out

@@ -1 +1 @@
-{"path":"github.com/tkrop/go-make","repo":"[email protected]:tkrop/go-make","build":"0001-01-01T00:00:00Z","commit":"0001-01-01T00:00:00Z","dirty":true,"go":"1.21.4","platform":"linux/amd64","compiler":"gc"}
+{"path":"github.com/tkrop/go-make","repo":"[email protected]:tkrop/go-make","build":"0001-01-01T00:00:00Z","commit":"0001-01-01T00:00:00Z","dirty":true,"go":"{{GOVERSION}}","platform":"{{PLATFORM}}","compiler":"{{COMPILER}}"}

internal/make/make_test.go

@@ -8,6 +8,7 @@ import (
 	"os"
 	"os/exec"
 	"regexp"
+	"runtime"
 	"strings"
 	"testing"
 
@@ -807,11 +808,21 @@ var (
 	// build path dependent parts.
 	regexMatchSourceDir = regexp.MustCompile( //nolint:gosimple // Just wrong!
 		"(?m)(['\\[])([^'\\]]*/)(go-make/[^'\\]]*)(['\\]])")
+
+	replacerFixture = strings.NewReplacer(
+		"{{GOVERSION}}", runtime.Version()[2:],
+		"{{PLATFORM}}", runtime.GOOS+"/"+runtime.GOARCH,
+		"{{COMPILER}}", runtime.Compiler)
 )
 
 func FilterMakeOutput(str string) string {
 	str = regexMatchTestDir.ReplaceAllString(str, "")
-	return regexMatchSourceDir.ReplaceAllString(str, "$1$3$4")
+	str = regexMatchSourceDir.ReplaceAllString(str, "$1$3$4")
+	return str
+}
+
+func SetupMakeFixture(str string) string {
+	return replacerFixture.Replace(str)
 }
 
 type MakeExecParams struct {
@@ -860,6 +871,9 @@ var testMakeExecParams = map[string]MakeExecParams{
 	},
 }
 
+// TODO: this test is sensitive to the execution parameters and fails if the
+// make target is started with option `--trace`. We need to figure out how this
+// influences the execution of the test and the output.
 func TestMakeExec(t *testing.T) {
 	SetupMakeConfig(t, "../..")
 
@@ -875,9 +889,9 @@ func TestMakeExec(t *testing.T) {
 
 			// Then
 			assert.Equal(t, param.expectExit, exit)
-			assert.Equal(t, param.expectStdout,
+			assert.Equal(t, SetupMakeFixture(param.expectStdout),
 				FilterMakeOutput(stdout.String()))
-			assert.Equal(t, param.expectStderr,
+			assert.Equal(t, SetupMakeFixture(param.expectStderr),
 				FilterMakeOutput(stderr.String()))
 		})
 }