Build tests with bsb and move them out of jscomp

Author: cknitt

.github/workflows/ci.yml

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

.gitignore

@@ -51,27 +51,8 @@ coverage
 .ninja_log
 .bsdeps
 .bsbuild
-lib/prebuilt.ninja
-lib/release.ninja
-lib/build.ninja
-lib/copy.ninja
-jscomp/release.ninja
-jscomp/build.ninja
-jscomp/compiler.ninja
-jscomp/runtime/build.ninja
-jscomp/others/build.ninja
-jscomp/stdlib-406/build.ninja
-jscomp/napkin.ninja
-jscomp/compilerEnv.ninja
-env.ninja
-jscomp/cppoVendor.ninja
-jscomp/cppoEnv.ninja
-jscomp/bin/bsb
-jscomp/bin/bsc
 lib/ocaml
-lib/bs
-lib/js/jscomp/test
-jscomp/build_tests/*/lib/
+tests/build_tests/*/lib/
 #ignore temporary directory
 *.goog.js
 *.jsoo.js

.istanbul.yml

@@ -4,7 +4,7 @@ instrumentation:
   extensions:
     - .js
   default-excludes: false
-  excludes: ["**/node_modules/**", "jscomp/test/**"]
+  excludes: ["**/node_modules/**", "tests/tests/**"]
   embed-source: false
   variable: __coverage__
   compact: true

CONTRIBUTING.md

@@ -146,26 +146,26 @@ make test
 
 **Run Mocha tests only (for our runtime code):**
 
-This will run our `mocha` unit test suite defined in `jscomp/test`.
+This will run our `mocha` unit test suite defined in `tests/tests`.
 
 ```
-node scripts/ciTest.js -mocha
+node scripts/test.js -mocha
 ```
 
 **Run build system test (integration tests):**
 
-This will run the whole build system test suite defined in `jscomp/build_tests`.
+This will run the whole build system test suite defined in `tests/build_tests`.
 
 ```
-node scripts/ciTest.js -bsb
+node scripts/test.js -bsb
 ```
 
 **Run ounit tests:**
 
 This will run unit tests for compiler related modules. The tests can be found in `jscomp/ounit_tests`.
 
 ```
-node scripts/ciTest.js -ounit
+node scripts/test.js -ounit
 ```
 
 ## Contributing to the Runtime

Makefile

@@ -30,7 +30,7 @@ node_modules/.bin/semver:
 	npm install
 
 test: lib
-	node scripts/ciTest.js -all
+	node scripts/test.js -all
 
 test-syntax:
 	bash ./scripts/test_syntax.sh
@@ -43,7 +43,7 @@ test-syntax-roundtrip:
 	bash ./scripts/testok.sh
 
 test-gentype:
-	make -C jscomp/gentype_tests/typescript-react-example clean test
+	make -C tests/gentype_tests/typescript-react-example clean test
 
 test-all: test test-gentype
 
@@ -79,7 +79,7 @@ checkformat:
 	bash scripts/format_check.sh
 
 clean-gentype:
-	make -C jscomp/gentype_tests/typescript-react-example clean
+	make -C tests/gentype_tests/typescript-react-example clean
 
 clean-rewatch:
 	cargo clean --manifest-path rewatch/Cargo.toml && rm -f rewatch/rewatch

biome.json

@@ -21,9 +21,9 @@
     "lineWidth": 80,
     "attributePosition": "auto",
     "ignore": [
-      "jscomp/build_tests/**/lib/**",
-      "jscomp/build_tests/**/src/**",
-      "jscomp/test/**",
+      "tests/build_tests/**",
+      "tests/tests/**",
+      "tests/tests_esmodule/**",
       "lib/**",
       "ninja/**",
       "playground/**",

bsconfig.json

@@ -131,10 +131,6 @@ let bs_package_output = "-bs-package-output"
 
 [@@@warning "+9"]
 
-(* Assume input is valid
-    coordinate with command line flag
-    {[ -bs-package-output commonjs:lib/js/jscomp/test:.js ]}
-*)
 let package_flag ({ format; in_source; suffix } : Bsb_spec_set.spec) dir =
   Ext_string.inter2 bs_package_output
     (Ext_string.concat5 (string_of_format format) Ext_string.single_colon

jscomp/bsb/bsb_package_specs.ml

@@ -29,9 +29,6 @@ val from_map : cwd:string -> Ext_json_types.t Map_string.t -> t
 val get_list_of_output_js : t -> string -> string list
 
 val package_flag_of_package_specs : t -> dirname:string -> string
-(**
-   Sample output: {[ -bs-package-output commonjs:lib/js/jscomp/test]}
-*)
 
 (* used to ensure each dir does exist *)
 val list_dirs_by : t -> (string -> unit) -> unit

jscomp/bsb/bsb_package_specs.mli

@@ -106,7 +106,7 @@ let oc_cmi buf namespace source =
      When ns is turned on, `B` is interprted as `Ns-B` which is a cyclic dependency,
      it can be errored out earlier
 
-   #5368: It turns out there are many false positives on detecting self-cycles (see: `jscomp/build_tests/zerocycle`)
+   #5368: It turns out there are many false positives on detecting self-cycles (see: `tests/build_tests/zerocycle`)
           To properly solve this, we would need to `jscomp/ml/depend.ml` because
             cmi and cmj is broken in the first place (same problem as in ocaml/ocaml#4618).
           So we will just ignore the self-cycles. Even if there is indeed a self-cycle, it should fail to compile anyway.