Run build_tests on Windows (rescript-lang#7065)

* Run build_tests on Windows

* CHANGELOG

Author: cknitt

.github/workflows/ci.yml

@@ -326,17 +326,12 @@ jobs:
         run: ./scripts/prebuilt.js
 
       - name: Run tests
-        if: runner.os != 'Windows'
         run: node scripts/test.js -all
 
       - name: Run gentype tests
         if: runner.os != 'Windows'
         run: make -C tests/gentype_tests/typescript-react-example clean test
 
-      - name: Run tests (Windows)
-        if: runner.os == 'Windows'
-        run: node scripts/test.js -mocha -theme -format
-
       - name: Build playground compiler
         if: matrix.build_playground
         run: |

CHANGELOG.md

@@ -46,6 +46,7 @@
 - Remove uncurried flag from bsb. https://github.com/rescript-lang/rescript-compiler/pull/7049
 - Build runtime/stdlib files with rescript/bsb instead of ninja.js. https://github.com/rescript-lang/rescript-compiler/pull/7063
 - Build tests with bsb and move them out of jscomp. https://github.com/rescript-lang/rescript-compiler/pull/7068
+- Run `build_tests` on Windows. https://github.com/rescript-lang/rescript-compiler/pull/7065
 
 # 12.0.0-alpha.3
 

scripts/test.js

@@ -2,7 +2,7 @@
 const cp = require("child_process");
 const path = require("path");
 const fs = require("fs");
-var { rescript_exe } = require("#cli/bin_path");
+const { rescript_exe } = require("#cli/bin_path");
 
 const duneBinDir = require("./dune").duneBinDir;
 
@@ -50,9 +50,13 @@ async function runTests() {
   }
 
   if (ounitTest) {
-    cp.execSync(path.join(duneBinDir, "ounit_tests"), {
-      stdio: [0, 1, 2],
-    });
+    if (process.platform === "win32") {
+      console.log("Skipping OUnit tests on Windows");
+    } else {
+      cp.execSync(path.join(duneBinDir, "ounit_tests"), {
+        stdio: [0, 1, 2],
+      });
+    }
   }
 
   if (mochaTest) {

tests/build_tests/case/input.js

@@ -1,14 +1,18 @@
 var p = require("child_process");
 var assert = require("assert");
 var { rescript_exe } = require("#cli/bin_path");
+var { normalizeNewlines } = require("../utils.js");
+
 var o = p.spawnSync(rescript_exe, { encoding: "utf8", cwd: __dirname });
 
 if (
   ![
     `Error: Invalid bsconfig.json implementation and interface have different path names or different cases src/demo vs src/Demo\n`,
-    // On linux files are parsed in different order
+    // Windows: path separator
+    `Error: Invalid bsconfig.json implementation and interface have different path names or different cases src\\demo vs src\\Demo\n`,
+    // Linux: files are parsed in different order
     `Error: Invalid bsconfig.json implementation and interface have different path names or different cases src/Demo vs src/demo\n`,
-  ].includes(o.stderr)
+  ].includes(normalizeNewlines(o.stderr))
 ) {
   assert.fail(o.stderr);
 }

tests/build_tests/case2/input.js

@@ -1,14 +1,18 @@
 var p = require("child_process");
 var assert = require("assert");
 var { rescript_exe } = require("#cli/bin_path");
+var { normalizeNewlines } = require("../utils.js");
+
 var o = p.spawnSync(rescript_exe, { encoding: "utf8", cwd: __dirname });
 
 if (
   ![
     `Error: Invalid bsconfig.json implementation and interface have different path names or different cases src/X vs src/x\n`,
-    // On linux files are parsed in different order
+    // Windows: path separator
+    `Error: Invalid bsconfig.json implementation and interface have different path names or different cases src\\X vs src\\x\n`,
+    // Linux: files are parsed in different order
     `Error: Invalid bsconfig.json implementation and interface have different path names or different cases src/x vs src/X\n`,
-  ].includes(o.stderr)
+  ].includes(normalizeNewlines(o.stderr))
 ) {
   assert.fail(o.stderr);
 }

tests/build_tests/cli_compile_status/input.js

@@ -1,27 +1,31 @@
 // @ts-check
 
 const assert = require("assert");
+const path = require("path");
 const child_process = require("child_process");
+const { normalizeNewlines } = require("../utils.js");
+
+const rescriptPath = path.join(__dirname, "..", "..", "..", "rescript")
 
 // Shows compile time for `rescript build` command
-let out = child_process.spawnSync(`../../../rescript`, ["build"], {
+let out = child_process.spawnSync("node", [rescriptPath, "build"], {
   encoding: "utf8",
   cwd: __dirname,
 });
 assert.match(
-  out.stdout,
+  normalizeNewlines(out.stdout),
   new RegExp(`>>>> Start compiling
 Dependency Finished
 >>>> Finish compiling \\d+ mseconds`),
 );
 
 // Shows compile time for `rescript` command
-out = child_process.spawnSync(`../../../rescript`, {
+out = child_process.spawnSync("node", [rescriptPath], {
   encoding: "utf8",
   cwd: __dirname,
 });
 assert.match(
-  out.stdout,
+  normalizeNewlines(out.stdout),
   new RegExp(`>>>> Start compiling
 Dependency Finished
 >>>> Finish compiling \\d+ mseconds`),
@@ -31,13 +35,10 @@ Dependency Finished
 // Because we can't be sure that -verbose is a valid argument
 // And bsb won't fail with a usage message.
 // It works this way not only for -verbose, but any other arg, including -h/--help/-help
-out = child_process.spawnSync(`../../../rescript`, ["build", "-verbose"], {
+out = child_process.spawnSync("node", [rescriptPath, "build", "-verbose"], {
   encoding: "utf8",
   cwd: __dirname,
 });
-assert.match(
-  out.stdout,
-  new RegExp(
-    `Package stack: test  \nDependency Finished\nninja.exe -C lib/bs \n`,
-  ),
-);
+
+assert.match(normalizeNewlines(out.stdout), /Package stack: test  \nDependency Finished\n/);
+assert.match(normalizeNewlines(out.stdout), /ninja.exe"? -C lib[\\/]bs ?\n/);