testutil: remove unnecessary formatting on err (#10)

* testutil: remove unnecessary formatting on err

Error can contain stuff like %s and then that gets interpreted as a
format string command. That's really not the intention here.

Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com>

* .github/workflows: try fixing

Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com>

---------

Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com>

Author: GiedriusS

.github/workflows/go.yaml

@@ -20,10 +20,14 @@ jobs:
         with:
           go-version: 1.19.x
 
-      - uses: actions/cache@v1
+      - uses: actions/cache@v4
         with:
-          path: ~/go/pkg/mod
+          path: |
+            ~/.cache/go-build
+            ~/go/pkg/mod
           key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }}
+          restore-keys: |
+            ${{ runner.os }}-go-
 
       - name: Linting & vetting.
         env:

testutil/testutil.go

@@ -54,7 +54,7 @@ func Ok(tb testing.TB, err error, v ...interface{}) {
 	if len(v) > 0 {
 		msg = fmt.Sprintf(v[0].(string), v[1:]...)
 	}
-	tb.Fatalf("\033[31m%s:%d: \"%s\"\n\n unexpected error: %s\033[39m\n\n", filepath.Base(file), line, withLimitf(msg), withLimitf(err.Error()))
+	tb.Fatalf("\033[31m%s:%d: \"%s\"\n\n unexpected error: %s\033[39m\n\n", filepath.Base(file), line, withLimitf(msg), err.Error())
 }
 
 // NotOk fails the test if an err is nil.