test: make tests work on Windows, add Windows to CI

.gitattributes

@@ -0,0 +1 @@
+**/testdata/fuzz/** binary

.github/workflows/test.yml

@@ -3,12 +3,10 @@ name: Test
 jobs:
   test:
     strategy:
+      fail-fast: false
       matrix:
         go-version: [1.17.x, 1.18.x]
-        # TODO(mvdan): fix and enable windows
-        # TODO(mvdan): mac seems to timeout too often
-        # os: [ubuntu-latest, macos-latest, windows-latest]
-        os: [ubuntu-latest, macos-latest]
+        os: [ubuntu-latest, macos-latest, windows-latest]
     runs-on: ${{ matrix.os }}
     steps:
     - name: Install Go

schema/dmt/roundtrip_test.go

@@ -3,6 +3,7 @@ package schemadmt_test
 import (
 	"bytes"
 	"io/ioutil"
+	"regexp"
 	"strings"
 	"testing"
 
@@ -30,6 +31,8 @@ func TestRoundtripSchemaSchema(t *testing.T) {
 func testRoundtrip(t *testing.T, want string, updateFn func(string)) {
 	t.Helper()
 
+	crre := regexp.MustCompile(`\r?\n`)
+	want = crre.ReplaceAllString(want, "\n")
 	nb := schemadmt.Type.Schema.Representation().NewBuilder()
 	err := ipldjson.Decode(nb, strings.NewReader(want))
 	qt.Assert(t, err, qt.IsNil)

schema/dsl/parse_test.go

@@ -5,6 +5,7 @@ import (
 	"flag"
 	"io/ioutil"
 	"path/filepath"
+	"regexp"
 	"strings"
 	"testing"
 
@@ -76,9 +77,7 @@ func TestParse(t *testing.T) {
 			err = yaml.Unmarshal(data, &fixt)
 			qt.Assert(t, err, qt.IsNil)
 
-			inJSON := strings.Replace(fixt.Expected, "  ", "\t", -1)
-
-			testParse(t, fixt.Schema, inJSON, func(updated string) {
+			testParse(t, fixt.Schema, fixt.Expected, func(updated string) {
 				updated = strings.Replace(updated, "\t", "  ", -1)
 				fixt.Expected = updated
 
@@ -97,6 +96,10 @@ func TestParse(t *testing.T) {
 func testParse(t *testing.T, inSchema, inJSON string, updateFn func(string)) {
 	t.Helper()
 
+	inJSON = strings.Replace(inJSON, "  ", "\t", -1) // fix non-tab indenting
+	crre := regexp.MustCompile(`\r?\n`)
+	inJSON = crre.ReplaceAllString(inJSON, "\n") // fix windows carriage-return
+
 	sch, err := schemadsl.ParseBytes([]byte(inSchema))
 	qt.Assert(t, err, qt.IsNil)
 

schema/gen/go/externUtil.go

@@ -4,7 +4,7 @@ import (
 	"go/ast"
 	"go/parser"
 	"go/token"
-	"path"
+	path "path/filepath"
 	"strings"
 )
 

schema/gen/go/testEngine_disabled_test.go

@@ -1,5 +1,5 @@
-//go:build skipgenbehavtests
-// +build skipgenbehavtests
+//go:build skipgenbehavtests || windows
+// +build skipgenbehavtests windows
 
 package gengo
 

schema/gen/go/testEngine_plugin_test.go

@@ -1,5 +1,5 @@
-//go:build cgo && !skipgenbehavtests
-// +build cgo,!skipgenbehavtests
+//go:build cgo && !skipgenbehavtests && !windows
+// +build cgo,!skipgenbehavtests,!windows
 
 package gengo
 

traversal/selector/spec_test.go

@@ -3,7 +3,9 @@ package selector_test
 import (
 	"bytes"
 	"io"
+	"io/ioutil"
 	"os"
+	"regexp"
 	"testing"
 
 	qt "github.com/frankban/quicktest"
@@ -25,7 +27,11 @@ func TestSpecFixtures(t *testing.T) {
 }
 
 func testOneSpecFixtureFile(t *testing.T, filename string) {
-	doc, err := testmark.ReadFile(filename)
+	data, err := ioutil.ReadFile(filename)
+	crre := regexp.MustCompile(`\r?\n`)
+	data = []byte(crre.ReplaceAllString(string(data), "\n")) // fix windows carriage-return
+
+	doc, err := testmark.Parse(data)
 	if os.IsNotExist(err) {
 		t.Skipf("not running spec suite: %s (did you clone the submodule with the data?)", err)
 	}