fix: windows file paths separator

Author: g4s8

.github/workflows/go.yml

@@ -24,7 +24,7 @@ jobs:
       - name: Build
         run: go build -v ./...
       - name: Test
-        run: go test -v ./...
+        run: go test -v "-test.v=true" ./...
       - name: Check examples
         run: |
           ./_examples/clean.sh

ast/file_test.go

@@ -2,12 +2,13 @@ package ast
 
 import (
 	"go/ast"
+	"path/filepath"
 	"testing"
 )
 
 func TestFileVisitor(t *testing.T) {
 	fset, pkg, docs := loadTestFileSet(t)
-	fh, fv, file := testFileVisitor(fset, pkg, "testdata/onetype.go", docs)
+	fh, fv, file := testFileVisitor(t, fset, pkg, filepath.Join("testdata", "onetype.go"), docs)
 	ast.Walk(fv, file)
 
 	types := make([]*TypeSpec, 0)

ast/parser_test.go

@@ -3,7 +3,6 @@ package ast
 import (
 	"fmt"
 	"os"
-	"path"
 	"path/filepath"
 	"strings"
 	"testing"
@@ -14,7 +13,7 @@ import (
 )
 
 func TestDataParser(t *testing.T) {
-	files, err := filepath.Glob("testdata/parser/*.txtar")
+	files, err := filepath.Glob(filepath.Join("testdata", "parser", "*.txtar"))
 	if err != nil {
 		t.Fatalf("failed to list testdata files: %s", err)
 	}
@@ -23,21 +22,26 @@ func TestDataParser(t *testing.T) {
 		t.Fatal("no testdata files found")
 	}
 
+	// normalize files
+	for i, file := range files {
+		files[i] = filepath.Clean(file)
+	}
+
 	for _, file := range files {
 		file := file
 
 		t.Run(filepath.Base(file), func(t *testing.T) {
-			t.Parallel()
+			// TODO: enable parallel tests after fixing #43
+			// t.Parallel()
 
+			t.Logf("Parse txtar file: %q", file)
 			ar, err := txtar.ParseFile(file)
 			if err != nil {
 				t.Fatalf("failed to parse txtar file: %s", err)
 			}
 
 			dir := t.TempDir()
-			if err := extractTxtar(ar, dir); err != nil {
-				t.Fatalf("failed to extract txtar: %s", err)
-			}
+			extractTxtar(t, ar, dir)
 
 			tc := readTestCase(t, dir)
 			testParser(t, dir, tc)
@@ -305,25 +309,26 @@ func checkTypeRef(t *testing.T, prefix string, expect, res *FieldTypeRef) {
 	}
 }
 
-//---
+func extractTxtar(t *testing.T, ar *txtar.Archive, dir string) {
+	t.Helper()
 
-func extractTxtar(ar *txtar.Archive, dir string) error {
+	t.Logf("Extracting txtar to %q", dir)
 	for _, file := range ar.Files {
 		name := filepath.Join(dir, file.Name)
+		t.Logf("Extracting %q to %q", file.Name, name)
 		if err := os.MkdirAll(filepath.Dir(name), 0o777); err != nil {
-			return err
+			t.Fatalf("failed to create dir: %s", err)
 		}
 		if err := os.WriteFile(name, file.Data, 0o666); err != nil {
-			return err
+			t.Fatalf("failed to write file: %s", err)
 		}
 	}
-	return nil
 }
 
 func readTestCase(t *testing.T, dir string) parserTestCase {
 	t.Helper()
 
-	testCaseFile, err := os.Open(path.Join(dir, "testcase.yaml"))
+	testCaseFile, err := os.Open(filepath.Join(dir, "testcase.yaml"))
 	if err != nil {
 		t.Fatalf("failed to open testcase file: %s", err)
 	}

ast/testhelper.go

@@ -5,6 +5,7 @@ import (
 	"go/doc"
 	"go/parser"
 	"go/token"
+	"testing"
 )
 
 type T interface {
@@ -98,10 +99,16 @@ func (h *testFileHandler) onFile(f *FileSpec) interface {
 }
 
 //nolint:staticcheck
-func testFileVisitor(fset *token.FileSet, pkg *ast.Package, fileName string,
+func testFileVisitor(t *testing.T,
+	fset *token.FileSet, pkg *ast.Package, fileName string,
 	docs *doc.Package,
 ) (*testFileHandler, *fileVisitor, *ast.File) {
+	t.Helper()
+
 	fileAst := pkg.Files[fileName]
+	if fileAst == nil {
+		t.Fatalf("file %q not found", fileName)
+	}
 	fileTkn := fset.File(fileAst.Pos())
 	fileSpec := &FileSpec{
 		Name: fileTkn.Name(),

generator_test.go

@@ -5,7 +5,6 @@ import (
 	"bytes"
 	"io"
 	"os"
-	"path"
 	"path/filepath"
 	"strings"
 	"testing"
@@ -53,7 +52,7 @@ func TestGenerator(t *testing.T) {
 			var out bytes.Buffer
 			runGenerator(t, gen, spec, dir, &out)
 
-			expectFile, err := os.Open(path.Join(dir, "expect.txt"))
+			expectFile, err := os.Open(filepath.Join(dir, "expect.txt"))
 			if err != nil {
 				t.Fatalf("failed to open expect.txt: %s", err)
 			}