feat(ci): add address sanitizer workflow

.github/workflows/asan.yml

@@ -0,0 +1,43 @@
+name: Address Sanitizer
+
+on:
+    push:
+
+jobs:
+    build-and-test:
+        runs-on: ubuntu-latest
+
+        steps:
+            - uses: actions/checkout@v3
+
+            - name: Setup node
+              uses: actions/setup-node@v3.6.0
+              with:
+                  node-version: 18
+                  check-latest: true
+                  cache: 'npm'
+
+            - name: Install dependencies
+              run: npm ci
+
+            - name: Run tests with address sanitizer
+              uses: addnab/docker-run-action@v3
+              with:
+                  registry: gcr.io
+                  image: ghcr.io/markusjx/prebuilt-debug-jdk:17-bullseye
+                  options: -v ${{ github.workspace }}:/app
+                  run: |
+                      curl -fsSL https://deb.nodesource.com/setup_18.x | bash - >> /dev/null
+                      apt-get install -yqq nodejs
+                      cargo update
+                      npm run build:napi:debug -- --target=x86_64-unknown-linux-gnu \
+                        --cargo-flags="-Zbuild-std"
+                      npm run postbuild
+
+                      export LD_PRELOAD=$(clang -print-file-name=libclang_rt.asan-x86_64.so)
+                      export RUST_BACKTRACE=1
+                      export FORCE_RUN_ALL_TESTS=true
+                      export INCREASE_TIMEOUT=true
+                      export SKIP_CLASS_TEST=true
+                      export DEBUG='napi:*'
+                      npm run testOnly

.github/workflows/check-style.yml

@@ -1,5 +1,5 @@
 name: Check-style
-on: [push, pull_request]
+on: [push]
 
 jobs:
     check-style:

.github/workflows/pages.yml

@@ -36,7 +36,7 @@ jobs:
               with:
                   node-version: 16.x
             - name: Rust Cache
-              uses: Swatinem/rust-cache@v2.0.0
+              uses: Swatinem/rust-cache@v2.2.1
             - name: Build docs
               run: |
                   npm ci

.github/workflows/test.yml

@@ -2,7 +2,6 @@ name: Test
 
 on:
     push:
-    pull_request:
 
 env:
     CARGO_TERM_COLOR: always
@@ -14,7 +13,7 @@ jobs:
         steps:
             - uses: actions/checkout@v3
             - name: Rust Cache
-              uses: Swatinem/rust-cache@v2.0.0
+              uses: Swatinem/rust-cache@v2.2.1
             - name: Build
               run: cargo build --verbose
             - name: Run tests
@@ -53,7 +52,7 @@ jobs:
         steps:
             - uses: actions/checkout@v3
             - name: Setup Node.js ${{matrix.node-version}} environment
-              uses: actions/setup-node@v3.4.1
+              uses: actions/setup-node@v3.6.0
               with:
                   node-version: ${{matrix.node-version}}
                   architecture: ${{matrix.arch}}
@@ -64,7 +63,12 @@ jobs:
                   java-version: '17'
                   architecture: ${{matrix.arch}}
             - name: Rust Cache
-              uses: Swatinem/rust-cache@v2.0.0
+              uses: Swatinem/rust-cache@v2.2.1
+            - name: Cache java definitions
+              uses: actions/cache@v3
+              with:
+                  path: '**/test/javaDefinitions'
+                  key: ${{runner.os}}-${{matrix.node-version}}-${{matrix.arch}}-javaDefinitions
             - name: Install
               run: npm ci
             - name: Build
@@ -78,6 +82,7 @@ jobs:
             - name: Run tests
               run: npm run testOnly
               timeout-minutes: 20
+
             - name: Create test summary
               uses: test-summary/action@dist
               if: always()
@@ -106,7 +111,7 @@ jobs:
         steps:
             - uses: actions/checkout@v3
             - name: Setup Node.js ${{matrix.node-version}} environment
-              uses: actions/setup-node@v3.4.1
+              uses: actions/setup-node@v3.6.0
               with:
                   node-version: ${{matrix.node-version}}
             - name: Setup java 17
@@ -122,7 +127,7 @@ jobs:
                   override: true
                   target: aarch64-unknown-linux-gnu
             - name: Rust Cache
-              uses: Swatinem/rust-cache@v2.0.0
+              uses: Swatinem/rust-cache@v2.2.1
             - name: Install cross compile toolchain
               run: |
                   sudo apt-get update
@@ -138,9 +143,6 @@ jobs:
               run: npm run build -- -- --target aarch64-unknown-linux-gnu
             - name: Install test dependencies
               run: npm install
-            # npm config set supportedArchitectures.cpu "arm64"
-            # npm config set supportedArchitectures.libc "glibc"
-            # npm install
             - name: Init qemu
               run: docker run --rm --privileged multiarch/qemu-user-static --reset -p yes
             - name: Run tests
@@ -179,15 +181,15 @@ jobs:
         steps:
             - uses: actions/checkout@v3
             - name: Setup Node.js environment
-              uses: actions/setup-node@v3.4.1
+              uses: actions/setup-node@v3.6.0
               with:
                   node-version: 16.x
             - uses: actions/setup-java@v3
               with:
                   distribution: 'temurin'
                   java-version: '17'
             - name: Rust Cache
-              uses: Swatinem/rust-cache@v2.0.0
+              uses: Swatinem/rust-cache@v2.2.1
             - name: Install
               run: npm ci
             - name: Run benchmarks

.gitignore

@@ -202,4 +202,5 @@ test.js
 Dockerfile
 test/javaDefinitions/**
 /testResults.xml
-test/testUtil.js
+test/testUtil.js
+testOut/

.prettierignore

@@ -317,4 +317,5 @@ Temporary Items
 
 docs/
 test/javaDefinitions/**
-/test/system_test/javaDefinitions/
+/test/system_test/javaDefinitions/
+testOut/

package.json

@@ -1,6 +1,6 @@
 {
     "name": "java-bridge",
-    "version": "2.2.3-beta.3",
+    "version": "2.2.3",
     "main": "dist/index.prod.min.js",
     "types": "dist/ts-src/index.d.ts",
     "description": "A bridge between Node.js and Java APIs",
@@ -78,6 +78,7 @@
     "devDependencies": {
         "@napi-rs/cli": "^2.14.8",
         "@types/chai": "^4.3.4",
+        "@types/folder-hash": "^4.0.2",
         "@types/glob": "^8.1.0",
         "@types/is-ci": "^3.0.0",
         "@types/mocha": "^10.0.1",
@@ -89,6 +90,7 @@
         "chai": "^4.3.7",
         "cpy-cli": "^4.2.0",
         "expose-gc": "^1.0.0",
+        "folder-hash": "^4.0.4",
         "is-ci": "^3.0.1",
         "mocha": "^10.2.0",
         "mocha-junit-reporter": "^2.2.0",

test/ClassTest.test.ts

@@ -3,6 +3,7 @@ import { expect } from 'chai';
 import { ClassTool, shouldIncreaseTimeout } from './testUtil';
 import path from 'path';
 
+const shouldSkip = process.env.SKIP_CLASS_TEST === 'true';
 const timeout = shouldIncreaseTimeout ? 60e3 : 20e3;
 let classTool: ClassTool | null = null;
 
@@ -35,6 +36,8 @@ function createJarWithBasicClass(
 }
 
 describe('ClassTest', () => {
+    if (shouldSkip) return;
+
     before(function () {
         this.timeout(timeout);
         if (!classTool) classTool = new ClassTool();
@@ -428,7 +431,7 @@ describe('ClassTest', () => {
 
         const ext = instance.getExtSync();
         expect(ext).to.be.an('object');
-    });
+    }).timeout(timeout);
 
     it('Class with external dependency loaded dynamically', () => {
         expect(() => importClass('dyn.importing.Class2')).to.throw();
@@ -450,7 +453,7 @@ describe('ClassTest', () => {
 
         const instance2 = ext.newInstanceSync();
         expect(instance2).to.be.an('object');
-    });
+    }).timeout(timeout);
 
     it('Class with properties', () => {
         const Class = importClass('ClassWithProperties');
@@ -475,7 +478,7 @@ describe('ClassTest', () => {
         expect(instance.s2).to.be.a('string');
         expect(instance.s2).to.equal('def');
         expect(() => (instance.s2 = 'def')).to.throw();
-    });
+    }).timeout(timeout);
 
     it('Class with complex properties', () => {
         const Class = importClass('ClassWithComplexProperties');
@@ -492,9 +495,10 @@ describe('ClassTest', () => {
         expect(instance.innerClass).to.have.property('s1');
         expect(instance.innerClass.s1).to.be.a('string');
         expect(instance.innerClass.s1).to.equal('abc');
-    });
+    }).timeout(timeout);
 
     after(function () {
+        this.timeout(timeout);
         try {
             classTool?.dispose();
         } catch (_) {}

test/ProxyTest.test.ts

@@ -29,7 +29,8 @@ declare class JavaString extends JavaClass {
 function getJavaVersion(): string {
     const version: string = importClass('java.lang.System')
         .getPropertySync('java.version')
-        .split('_')[0];
+        .split('_')[0]
+        .replace(/^(\d+)-internal$/gm, '$1.0.0');
 
     const arr = version.split('.');
     arr.length = Math.min(arr.length, 3);

test/TypescriptDefinitionGenerator.test.ts

@@ -3,7 +3,7 @@ import { expect } from 'chai';
 import ts from 'typescript';
 import path from 'path';
 import * as fs from 'fs';
-import { shouldIncreaseTimeout } from './testUtil';
+import { forceRunAllTests, shouldIncreaseTimeout } from './testUtil';
 import isCi from 'is-ci';
 
 interface Diagnostics {
@@ -126,7 +126,7 @@ describe('TypescriptDefinitionGenerator test', () => {
     }).timeout(timeoutMs);
 
     it("Generate 'java.io.FileOutputSteam' definitions", async function () {
-        if (isCi) {
+        if (isCi && !forceRunAllTests) {
             this.skip();
         }
 
@@ -145,5 +145,5 @@ describe('TypescriptDefinitionGenerator test', () => {
             FileOutputStream.nullOutputStreamSync()!.flushSync();
             `
         );
-    }).timeout(timeoutMs * 2);
+    }).timeout(timeoutMs * 4);
 });