From 0d040345a00770c2e97448ef88e382b9f70d61c9 Mon Sep 17 00:00:00 2001
From: Konstantin Burkalev <KSDaemon@gmail.com>
Date: Wed, 29 Oct 2025 15:52:58 +0200
Subject: [PATCH] chore(ci): Enable unit and integration tests for node.js v24

---
 .github/workflows/push.yml | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/push.yml b/.github/workflows/push.yml
index 0cd822e65f8dc..71e3a31995605 100644
--- a/.github/workflows/push.yml
+++ b/.github/workflows/push.yml
@@ -82,8 +82,7 @@ jobs:
     strategy:
       matrix:
         # Current docker version + next LTS
-        # TODO: Add 24 after it's been released
-        node-version: [22.x]
+        node-version: [22.x, 24.x]
         # Don't forget to update build-native-release
         python-version: [3.11]
         transpile-worker-threads: [false, true]
@@ -419,7 +418,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [22.x]
+        node-version: [22.x, 24.x]
         # Vertica tests are disabled because around 20.08.2025 someone
         # totally removed all vertica-ce docker repository from dockerhub.
         # @see https://github.com/vertica/vertica-containers/issues/64
@@ -523,13 +522,16 @@ jobs:
           DRIVERS_TESTS_SNOWFLAKE_CUBEJS_DB_USER: ${{ secrets.DRIVERS_TESTS_CUBEJS_DB_SNOWFLAKE_USER }}
           DRIVERS_TESTS_SNOWFLAKE_CUBEJS_DB_PASS: ${{ secrets.DRIVERS_TESTS_CUBEJS_DB_SNOWFLAKE_PASS }}
       - name: Fix lcov paths
+        if: (matrix.node-version == '24.x')
         run: |
           ./.github/actions/codecov-fix.sh
       - name: Combine all fixed LCOV files
+        if: (matrix.node-version == '24.x')
         run: |
           echo "" > ./combined-integration-${{ matrix.db }}-${{ matrix.use_tesseract_sql_planner }}.lcov
           find ./packages -type f -name lcov.fixed.info -exec cat {} + >> ./combined-integration-${{ matrix.db }}-${{ matrix.use_tesseract_sql_planner }}.lcov || true
       - name: Upload coverage artifact
+        if: (matrix.node-version == '24.x')
         uses: actions/upload-artifact@v4
         with:
           name: coverage-integration-${{ matrix.db }}-${{ matrix.use_tesseract_sql_planner }}