Upgraded bun & node on ci

Author: viktor-podzigun

.github/workflows/ci.yml

@@ -15,26 +15,26 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Git checkout
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           fetch-depth: 0
 
       - name: Setup Node.js
-        uses: actions/setup-node@v3
+        uses: actions/setup-node@v4
         with:
-          node-version: "20"
+          node-version: "24"
 
       - name: Cache node_modules
-        uses: actions/cache@v3
+        uses: actions/cache@v4
         with:
           path: |
             ~/.npm
             ~/.nvm
             ~/work/react-assert/react-assert/node_modules
             ~/work/react-assert/react-assert/package-lock.json
-          key: ${{ runner.os }}-node_modules-cache-v3-${{ hashFiles('package.json') }}
+          key: ${{ runner.os }}-node_modules-cache-v4-${{ hashFiles('package.json') }}
           restore-keys: |
-            ${{ runner.os }}-node_modules-cache-v3-
+            ${{ runner.os }}-node_modules-cache-v4-
 
       - name: Extract Tag Name
         run: echo "TAG_NAME=$(echo ${GITHUB_REF##*/})" >> $GITHUB_ENV

.github/workflows/ci_bun.yml

@@ -13,24 +13,24 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Git checkout
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           fetch-depth: 0
 
       - name: Setup Bun
         uses: oven-sh/setup-bun@v1
         with:
-          bun-version: "1.0.1"
+          bun-version: "1.2.0"
 
       - name: Cache Dependencies
-        uses: actions/cache@v3
+        uses: actions/cache@v4
         with:
           path: |
             ~/.bun/install/cache
             ~/work/react-assert/react-assert/bun.lockb
-          key: ${{ runner.os }}-bun-cache-v4-${{ hashFiles('package.json') }}
+          key: ${{ runner.os }}-bun-cache-v5-${{ hashFiles('package.json') }}
           restore-keys: |
-            ${{ runner.os }}-bun-cache-v4-
+            ${{ runner.os }}-bun-cache-v5-
 
       - name: Run tests
         run: |

src/assertComponent.mjs

@@ -32,29 +32,29 @@ function assertComponentImpl(path, result, expectedElement) {
   const pathName = path ? `${path} > ${name}` : Object(name).toString();
 
   if (typeof result === "string" || typeof expectedElement === "string") {
-    assert.deepEqual(
-      result,
-      expectedElement,
-      `Element doesn't match for ${pathName}` +
-        `\n\tactual:   ${result}` +
-        `\n\texpected: ${expectedElement}`
-    );
+    if (result !== expectedElement) {
+      assert.fail(
+        `Element doesn't match for ${pathName}` +
+          `\n\tactual:   ${result}` +
+          `\n\texpected: ${expectedElement}`
+      );
+    }
     return;
   }
 
-  assert.deepEqual(
-    result.type,
-    expectedElement.type,
-    `Component type doesn't match for ${pathName}` +
-      `\n\tactual:   ${
-        result.type.displayName ? result.type.displayName : result.type
-      }` +
-      `\n\texpected: ${
-        expectedElement.type.displayName
-          ? expectedElement.type.displayName
-          : expectedElement.type
-      }`
-  );
+  if (result.type !== expectedElement.type) {
+    assert.fail(
+      `Component type doesn't match for ${pathName}` +
+        `\n\tactual:   ${
+          result.type.displayName ? result.type.displayName : result.type
+        }` +
+        `\n\texpected: ${
+          expectedElement.type.displayName
+            ? expectedElement.type.displayName
+            : expectedElement.type
+        }`
+    );
+  }
 
   Object.keys(expectedElement.props)
     .filter((p) => {
@@ -84,13 +84,13 @@ function assertComponentImpl(path, result, expectedElement) {
       );
     }
   } else {
-    assert.deepEqual(
-      children.length,
-      expectedChildren.length,
-      `Children count doesn't match for ${pathName}` +
-        `\n\tactual:   ${children.length}` +
-        `\n\texpected: ${expectedChildren.length}`
-    );
+    if (children.length !== expectedChildren.length) {
+      assert.fail(
+        `Children count doesn't match for ${pathName}` +
+          `\n\tactual:   ${children.length}` +
+          `\n\texpected: ${expectedChildren.length}`
+      );
+    }
 
     expectedChildren.forEach((expected, i) => {
       assertComponentImpl(pathName, children[i], expected);

src/assertComponents.mjs

@@ -6,13 +6,13 @@ import assertComponent from "./assertComponent.mjs";
  * @param { (import('react').ReactElement | string)[] } expectedElements
  */
 function assertComponents(results, ...expectedElements) {
-  assert.deepEqual(
-    results.length,
-    expectedElements.length,
-    `Components count doesn't match` +
-      `\n\tactual:   ${results.length}` +
-      `\n\texpected: ${expectedElements.length}`
-  );
+  if (results.length !== expectedElements.length) {
+    assert.fail(
+      `Components count doesn't match` +
+        `\n\tactual:   ${results.length}` +
+        `\n\texpected: ${expectedElements.length}`
+    );
+  }
 
   expectedElements.forEach((expected, i) => {
     assertComponent(results[i], expected);

test/assertComponent.test.mjs

@@ -33,11 +33,13 @@ describe("assertComponent.test.mjs", () => {
     }
 
     //then
-    assert.deepEqual(
-      resError?.message,
-      "Attribute value doesn't match for p.testArr.0" +
-        "\n\tactual:   test1" +
-        "\n\texpected: test2"
+    assert.match(
+      resError?.message ?? "",
+      new RegExp(
+        "Attribute value doesn't match for p\\.testArr\\.0" +
+          "\n\tactual:   test1" +
+          "\n\texpected: test2"
+      )
     );
   });
 
@@ -62,11 +64,13 @@ describe("assertComponent.test.mjs", () => {
     }
 
     //then
-    assert.deepEqual(
-      resError?.message,
-      "Attribute value doesn't match for p.testObj.test" +
-        "\n\tactual:   1" +
-        "\n\texpected: 2"
+    assert.match(
+      resError?.message ?? "",
+      new RegExp(
+        "Attribute value doesn't match for p\\.testObj\\.test" +
+          "\n\tactual:   1" +
+          "\n\texpected: 2"
+      )
     );
   });
 
@@ -91,11 +95,13 @@ describe("assertComponent.test.mjs", () => {
     }
 
     //then
-    assert.deepEqual(
-      resError?.message,
-      "Attribute value doesn't match for p.testObj.test" +
-        "\n\tactual:   () => 1" +
-        "\n\texpected: undefined"
+    assert.match(
+      resError?.message ?? "",
+      new RegExp(
+        "Attribute value doesn't match for p\\.testObj\\.test" +
+          "\n\tactual:   \\(\\) => 1" +
+          "\n\texpected: undefined"
+      )
     );
   });
 
@@ -118,11 +124,13 @@ describe("assertComponent.test.mjs", () => {
     }
 
     //then
-    assert.deepEqual(
-      resError?.message,
-      "Attribute value doesn't match for p.disabled" +
-        "\n\tactual:   true" +
-        "\n\texpected: false"
+    assert.match(
+      resError?.message ?? "",
+      new RegExp(
+        "Attribute value doesn't match for p\\.disabled" +
+          "\n\tactual:   true" +
+          "\n\texpected: false"
+      )
     );
   });
 

test/assertComponents.test.mjs

@@ -1,9 +1,9 @@
 import React from "react";
 import TestRenderer from "react-test-renderer";
+import assert from "node:assert/strict";
 import { assertComponents } from "../index.mjs";
 import { TestComp, TestComp2 } from "./testComponents.mjs";
 
-import { strict as assert } from "node:assert";
 const { describe, it } = await (async () => {
   // @ts-ignore
   const module = process.isBun ? "bun:test" : "node:test";