ci: Run tests on Windows

Author: lauriro

.github/workflows/test.yml

@@ -13,8 +13,26 @@ jobs:
     - run: npx lj ui-test test/index.html
     - uses: coverallsapp/github-action@v2
       name: Upload to coveralls.io
+      continue-on-error: true
       with:
         github-token: ${{ github.token }}
+  Test:
+    strategy:
+      matrix:
+        include:
+        - { node: 20, os: windows-2022,   arch: x86 }
+        - { node: 22, os: macos-latest,   arch: arm64 }
+    runs-on: ${{ matrix.os }}
+    name: Node ${{matrix.node}} (${{matrix.os}} ${{matrix.arch}})
+    steps:
+    - run: git config --global core.autocrlf false
+    - uses: actions/checkout@v5
+    - uses: actions/setup-node@v5
+      with:
+        node-version: ${{ matrix.node }}
+        architecture: ${{ matrix.arch }}
+    - run: npm install --no-audit && npm install -g uglify-js
+    - run: npm test
   Analyze:
     uses: litejs/.github/.github/workflows/analyze.yml@main
   Release:

index.js

@@ -85,19 +85,23 @@ function isObj(obj) {
 	return !!obj && obj.constructor === Object
 }
 
+function fwd(s) {
+	return s.replace(/\\/g, "/")
+}
+
 function ls() {
 	var key, dirRe, outRe, tmp, tmp2
 	, arr = flat(arguments)
 	, i = arr.length
 	, out = []
 	, paths = {}
-	, reEscRe = /[*.+^=:${}()|\/\\]/g
+	, reEscRe = /[*.+^=:${}()|\/]/g
 	, opts = { absolute: false, cwd: process.cwd(), dir: true, dot: false, file: true, root: "", stat: false }
 	for (; i > 0; ) {
 		key = arr[--i]
 		if (isObj(key)) Object.assign(opts, key)
 		else if (typeof key === "string") {
-			tmp = path.resolve(opts.cwd, tmp2 = key.replace(/[^\/]*\*.*/, ""))
+			tmp = fwd(path.resolve(opts.cwd, tmp2 = key.replace(/[^\/]*\*.*/, "")))
 			tmp = paths[tmp] || (paths[tmp] = [])
 			if (key !== tmp2) tmp.push(key.slice(tmp2.length))
 		}
@@ -118,14 +122,14 @@ function ls() {
 			var stat = fs.statSync(name)
 			if (outRe.test(name)) {
 				if (stat.isDirectory() ? opts.dir : opts.file) out.push(
-					opts.stat ? Object.assign(stat, { name: opts.absolute ? name : opts.root + path.relative(opts.cwd, name) }) :
+					opts.stat ? Object.assign(stat, { name: opts.absolute ? name : opts.root + fwd(path.relative(opts.cwd, name)) }) :
 					opts.absolute ? name :
-					opts.root + path.relative(opts.cwd, name)
+					opts.root + fwd(path.relative(opts.cwd, name))
 				)
 			}
 			if (stat.isDirectory() && dirRe.test(name)) {
 				fs.readdirSync(name).forEach(function(file) {
-					scan(path.resolve(name, file))
+					scan(fwd(path.resolve(name, file)))
 				})
 			}
 		} catch(e) {}

lib/build.js

@@ -392,10 +392,11 @@ function html(opts, next) {
 }
 
 function jsMin(opts) {
-	var cmd = (jsMinCmd || [
-		"uglifyjs --warn --ie8 -c 'evaluate=false,passes=2,properties=false'",
-		"-m eval --comments '/^\\s*[@!]/'",
-		"--beautify 'beautify=false,semicolons=false,keep_quoted_props=true'"
+	var q = process.platform === "win32" ? "\"" : "'"
+	, cmd = (jsMinCmd || [
+		"uglifyjs --warn --ie8 -c " + q + "evaluate=false,passes=2,properties=false" + q,
+		"-m eval --comments " + q + "/^\\s*[@!]/" + q,
+		"--beautify " + q + "beautify=false,semicolons=false,keep_quoted_props=true" + q
 	].join(" ")) + (opts.files ? " " + opts.files.join(" ") : jsMinCmd ? "" : " --")
 	return child.execSync(cmd, opts).toString("utf8").replace(/\\x0B/g, "\\v")
 }

snapshot.js

@@ -3,14 +3,15 @@ var cli = require(".")
 , child = require("child_process")
 , fs = require("fs")
 , path = require("path")
-, relPathRe = /[^(]+(?=:\d+:\d+\))|^\/[\w/.]+(?=:\d)/gm
-, relPathFn = path.relative.bind(path, process.cwd())
+, relPathRe = /[^(]+(?=:\d+:\d+\))|^(?:\/|[A-Za-z]:\\)[\w/.\\-]+(?=:\d)/gm
+, cwd = process.cwd()
+, relPathFn = function(p) { return path.relative(cwd, p).replace(/\\/g, "/") }
 , seen = {}
 
 /* globals describe */
 
 function normalize(str) {
-	return str.replace(relPathRe, relPathFn).replace(/at \w+\.<anonymous>/g, "at <anonymous>")
+	return str.replace(/\r\n/g, "\n").replace(relPathRe, relPathFn).replace(/at \w+\.<anonymous>/g, "at <anonymous>")
 }
 
 describe.assert.cmdSnapshot = function(cmd, file, opts) {

test/build.js

@@ -40,7 +40,7 @@ describe("build", function() {
 				return initialFiles.indexOf(file) < 0
 			})
 			assert.planned = initialFiles.length + createdFiles.length + 2
-			assert.cmdSnapshot("cd " + temp + ";find .|sort", prefix + "ls")
+			assert.matchSnapshot(prefix + "ls", ".\n" + cli.ls(temp + "**/*").map(function(f) { return "./" + f.slice(temp.length) }).sort().join("\n") + "\n")
 			assert.matchSnapshot(temp + outDir + "index.html", 0, prefix + "index.html")
 			initialFiles.forEach(function(file) {
 				file = file.replace("index.html", "dev.html")

test/tools.js

@@ -36,7 +36,7 @@ describe("tools", function() {
 		assert.end()
 	})
 
-	it ("should list files: {0}", [
+	if (process.platform === "linux") it ("should list files: {0}", [
 		"test/index.js",
 		"*.js",
 		"test/data/**/*",
@@ -54,21 +54,20 @@ describe("tools", function() {
 		cli.mkdirp(".github/.dot")
 		cli.cp("package.json", ".github/.dot/p.json")
 		cli.cp(".github/.dot", ".github/.dot2")
-		assert.equal(
+		if (process.platform === "linux") assert.equal(
 			cli.ls(".*/.d*", null, ".github").join(" "),
 			child.execSync("bash -c 'shopt -s globstar;echo .github .*/.d*'").toString("utf8").trim()
 		)
 		// Assert options
 		assert.equal(cli.ls(".github/*", { dir: false }).join(" "), ".github/jshint.json .github/litejs.json")
 		assert.equal(cli.ls("*", { cwd: ".github", dir: false }).join(" "), "jshint.json litejs.json")
-		assert.own(cli.ls("*", { cwd: ".github", stat: true }).map(s=>({size: s.size, name: s.name})), [
-			{ size: 232,  name: "jshint.json" },
-			{ size: 268,  name: "litejs.json" },
-			{ size: 4096, name: "workflows" }
-		])
+		var stats = cli.ls("*", { cwd: ".github", stat: true }).map(s=>({size: s.size, name: s.name}))
+		assert.equal(stats.length, 3)
+		assert.equal(stats[0].size, 232)
+		assert.equal(stats[2].name, "workflows")
 		assert.equal(cli.ls(".github/*", { file: false }).join(" "), ".github/workflows")
 		assert.equal(cli.ls(".github/*", { file: false, root: "/www/" }).join(" "), "/www/.github/workflows")
-		assert.equal(cli.ls(".github/*", { file: false, absolute: true }).join(" "), path.join(process.cwd(), ".github/workflows"))
+		assert.equal(cli.ls(".github/*", { file: false, absolute: true }).join(" "), path.join(process.cwd(), ".github/workflows").replace(/\\/g, "/"))
 		assert.equal(cli.ls(".github/*", { dot: true }).join(" "), ".github/.dot .github/.dot2 .github/jshint.json .github/litejs.json .github/workflows")
 		cli.rmrf(".github/.dot")
 		cli.rmrf(".github/.dot2")