Merge pull request #2349 from wellwelwel/osx

ci: refactor CI - OSX

Author: wellwelwel

.github/workflows/ci-osx.yml

@@ -18,24 +18,11 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        node-version: [18.x, 20.x]
-        mysql-version: ["mysql:8.0.22", "mysql:8.0.33"]
+        node-version: [ 20.x]
+        mysql-version: ["mysql:8.0.33"]
         use-compression: [0, 1]
-        use-tls: [0]
+        use-tls: [0, 1]
         mysql_connection_url_key: [""]
-        # TODO - add mariadb to the matrix. currently few tests are broken due to mariadb incompatibilities
-        include:
-          # 20.x
-          - node-version: "20.x"
-            mysql-version: "mysql:8.0.33"
-            use-compression: 1
-            use-tls: 0
-            use-builtin-test-runner: 1
-          - node-version: "20.x"
-            mysql-version: "mysql:8.0.33"
-            use-compression: 0
-            use-tls: 1
-            use-builtin-test-runner: 1
     env:
       MYSQL_CONNECTION_URL: ${{ secrets[matrix.mysql_connection_url_key] }}
 
@@ -82,7 +69,3 @@ jobs:
 
       - name: Run tests
         run: FILTER=${{matrix.filter}} MYSQL_USE_TLS=${{ matrix.use-tls }} MYSQL_USE_COMPRESSION=${{ matrix.use-compression }} npm run coverage-test
-
-      - name: Run tests with built-in node test runner
-        if: ${{ matrix.use-builtin-test-runner }}
-        run: FILTER=${{matrix.filter}} MYSQL_USE_TLS=${{ matrix.use-tls }} MYSQL_USE_COMPRESSION=${{ matrix.use-compression }} npm run test:builtin-node-runner