From 47b63bed870bc2704314a52ce4c205ca7e6fa9cf Mon Sep 17 00:00:00 2001 From: satoren Date: Fri, 28 Jun 2024 00:06:20 +0000 Subject: [PATCH] add tests with ASAN to node CI --- .github/workflows/mediasoup-node.yaml | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/.github/workflows/mediasoup-node.yaml b/.github/workflows/mediasoup-node.yaml index b035e0c56f..462bbe6492 100644 --- a/.github/workflows/mediasoup-node.yaml +++ b/.github/workflows/mediasoup-node.yaml @@ -11,12 +11,22 @@ concurrency: jobs: ci: strategy: + fail-fast: false matrix: ci: - os: ubuntu-20.04 node: 18 - os: ubuntu-22.04 node: 20 + - os: ubuntu-24.04 + node: 20 + meson_args: '-Db_sanitize=address' + - os: ubuntu-24.04 + node: 20 + meson_args: '-Db_sanitize=undefined' + - os: ubuntu-24.04 + node: 20 + meson_args: '-Db_sanitize=thread' - os: macos-12 node: 18 - os: macos-14 @@ -33,6 +43,7 @@ jobs: MEDIASOUP_SKIP_WORKER_PREBUILT_DOWNLOAD: 'true' MEDIASOUP_LOCAL_DEV: 'true' MEDIASOUP_BUILDTYPE: ${{ matrix.build-type }} + MESON_ARGS: ${{ matrix.ci.meson_args }} steps: - name: Checkout @@ -54,6 +65,8 @@ jobs: - name: npm ci run: npm ci --foreground-scripts + env: + ASAN_OPTIONS: 'detect_leaks=0' # Disable leak detection because it is detected by the tool flatc used to build - name: npm run lint:node run: npm run lint:node