fix: fix the check for compat tests

Author: aminya

.github/workflows/CI.yml

@@ -179,15 +179,15 @@ jobs:
             pnpm run test.unit
         continue-on-error: true
 
-      - name: Test Compatibility 
+      - name: Test Compatibility
         if: ${{ !matrix.docker }}
         uses: nick-fields/retry@v3
         with:
           timeout_minutes: 5
           max_attempts: 1
           command: |
             pnpm run clean.temp
-            pnpm run test.compat
+            pnpm run test.unit.compat
         continue-on-error: true
 
       - name: Test Electron (Main)

.mocharc.js

@@ -3,7 +3,6 @@
  */
 const config = {
   require: ["ts-node/register"],
-  spec: ["test/unit/**/*-test.ts"],
   "expose-gc": true,
   "v8-expose-gc": true,
   exit: true,

.vscode/settings.json

@@ -18,6 +18,7 @@
   "mochaExplorer.env": {
     "INCLUDE_COMPAT_TESTS": "true"
   },
+  "mochaExplorer.files": "test/unit/**/*-test.ts",
   "files.exclude": {
     "**/.DS_Store": true,
     "**/Thumbs.db": true,

package.json

@@ -98,11 +98,11 @@
     "build.native.debug": "cmake-ts dev-os-only",
     "build": "run-p build.js build.native",
     "build.debug": "run-s build.js build.native.debug",
-    "test": "run-s clean.temp test.unit.compat clean.temp test.unit",
-    "test.unit": "cross-env INCLUDE_COMPAT_TESTS=false mocha ./test/unit/*-test.ts",
-    "test.unit.compat": "cross-env INCLUDE_COMPAT_TESTS=true mocha ./test/unit/compat/*-test.ts",
+    "test": "run-s test.unit",
+    "test.unit": "run-s clean.temp build && cross-env INCLUDE_COMPAT_TESTS=false mocha ./test/unit/*-test.ts",
+    "test.unit.compat": "run-s clean.temp build && cross-env INCLUDE_COMPAT_TESTS=true mocha ./test/unit/compat/*-test.ts",
     "test.smoke": "bash ./script/smoke-test.bash",
-    "test.skip_gc_tests": "run-s clean.temp build.debug && cross-env SKIP_GC_TESTS=true mocha",
+    "test.skip_gc_tests": "run-s clean.temp build && cross-env SKIP_GC_TESTS=true mocha",
     "test.electron.main": "run-s clean.temp build && electron-mocha",
     "format": "run-s format.prettier format.clang-format",
     "format.prettier": "prettier -l --cache --cache-location ./.cache/prettier --write .",

test/unit/compat/context-test.ts

@@ -1,7 +1,7 @@
 import * as zmq from "../../../v5-compat"
 import {assert} from "chai"
 
-if (process.env.INCLUDE_COMPAT_TESTS) {
+if (process.env.INCLUDE_COMPAT_TESTS === "true") {
   describe("compat context", function () {
     it("should support setting max io threads", function () {
       zmq.Context.setMaxThreads(3)

test/unit/compat/exports-test.ts

@@ -3,7 +3,7 @@ import {capability} from "../../../src"
 import {assert} from "chai"
 import semver from "semver"
 
-if (process.env.INCLUDE_COMPAT_TESTS) {
+if (process.env.INCLUDE_COMPAT_TESTS === "true") {
   describe("compat exports", function () {
     it("should export a valid version", function () {
       assert.ok(semver.valid(zmq.version))

test/unit/compat/gc-test.ts

@@ -2,7 +2,7 @@ import * as zmq from "../../../v5-compat"
 import {assert} from "chai"
 import {testProtos, uniqAddress} from "../helpers"
 
-if (process.env.INCLUDE_COMPAT_TESTS) {
+if (process.env.INCLUDE_COMPAT_TESTS === "true") {
   for (const proto of testProtos("tcp", "inproc")) {
     describe(`compat socket with ${proto}`, function () {
       let address: string

test/unit/compat/socket-error-callback-test.ts

@@ -3,7 +3,7 @@ import type {Socket} from "../../../v5-compat"
 import {isFullError} from "../../../src/errors"
 import {assert} from "chai"
 
-if (process.env.INCLUDE_COMPAT_TESTS) {
+if (process.env.INCLUDE_COMPAT_TESTS === "true") {
   describe("compat socket error callback", function () {
     let sock: Socket
 

test/unit/compat/socket-events-test.ts

@@ -2,7 +2,7 @@ import * as zmq from "../../../v5-compat"
 import {assert} from "chai"
 import {testProtos, uniqAddress} from "../helpers"
 
-if (process.env.INCLUDE_COMPAT_TESTS) {
+if (process.env.INCLUDE_COMPAT_TESTS === "true") {
   for (const proto of testProtos("tcp", "inproc")) {
     describe(`compat socket with ${proto} events`, function () {
       let address: string

test/unit/compat/socket-messages-test.ts

@@ -2,8 +2,8 @@ import * as zmq from "../../../v5-compat"
 import {assert} from "chai"
 import {testProtos, uniqAddress} from "../helpers"
 
-if (process.env.INCLUDE_COMPAT_TESTS) {
-  for (const proto of testProtos("tcp", "inproc")) {
+if (process.env.INCLUDE_COMPAT_TESTS === "true") {
+  for (const proto of testProtos( "inproc")) {
     describe(`compat socket with ${proto} messages`, function () {
       let push: zmq.Socket
       let pull: zmq.Socket