fix: do not initialize distro in unsupported NodeJS versions

.github/workflows/push-actions.yml

@@ -1,12 +1,55 @@
 name: Tracer Testing
 on: [push]
 jobs:
-  version-testing:
+
+  build-package:
+    name: Validation ${{ matrix.npm-task }}
     runs-on: ubuntu-latest
+    continue-on-error: true
     strategy:
+      fail-fast: false # We don't want linting to prevent unit testing
       matrix:
-        node_cimg_tag: [ 14, 16, 18 ]
-    name: Node ${{ matrix.node_cimg_tag }}
+        npm-task: ['prettier:ci', 'lint', 'test:unit']
+    steps:
+      - uses: actions/checkout@v3
+      - name: Set up Node
+        uses: actions/setup-node@v3
+        with:
+          node-version: 16
+          architecture: x64
+      - run: sudo apt-get update && sudo apt-get install build-essential libsdl-pango-dev libcairo2-dev libpango1.0-dev libjpeg-dev libgif-dev librsvg2-dev || true # Node.js 18 does not carry pango
+      - run: npm ci
+      - run: npm run ${{ matrix.npm-task }}
+      - name: Upload distro as artifact
+        uses: actions/upload-artifact@v3
+        if: ${{ matrix.npm-task == 'test:unit' }}
+        with:
+          name: distro-package
+          path: distro.tgz
+          retention-days: 5
+
+  instrumentations-to-test:
+    name: List instrumentations to test
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v3
+    - id: list_instrumentations
+      run: |
+        instrumentations_as_json_list=$(ls -1 test/instrumentations | jq --raw-input --slurp 'split("\n") | map(select(. != ""))' | jq -r tostring)
+        echo "instrumentations=${instrumentations_as_json_list}" >> "${GITHUB_OUTPUT}"
+        echo "instrumentations=${instrumentations_as_json_list}"
+    outputs:
+      instrumentations: ${{ steps.list_instrumentations.outputs.instrumentations }}
+
+  supported-node-versions-testing:
+    name: 'Node ${{ matrix.node_cimg_tag }} ${{ matrix.instrumentations_to_test }} tests'
+    runs-on: ubuntu-latest
+    needs: [build-package, instrumentations-to-test]
+    strategy:
+      fail-fast: false # We want to see all tests being tried, rather than stop when the first fails
+      matrix:
+        node_cimg_tag: [ 14, 16, 18, 20 ]
+        instrumentations_to_test: ${{ fromJson(needs.instrumentations-to-test.outputs.instrumentations) }}
     steps:
       - uses: actions/checkout@v3
       - name: Set up Node
@@ -16,7 +59,40 @@ jobs:
           architecture: x64
       - run: sudo apt-get update && sudo apt-get install build-essential libsdl-pango-dev libcairo2-dev libpango1.0-dev libjpeg-dev libgif-dev librsvg2-dev || true # Node.js 18 does not carry pango
       - run: npm ci
-      - run: npm run build
-      - run: npm run test:unit
-      - run: npm run test:component
-      - run: npm run test:integration:version
+      - run: npm run test:instrumentations
+        env:
+          INSTRUMENTATION_UNDER_TEST: ${{ matrix.instrumentations_to_test }}
+
+  unsupported-node-versions-testing:
+    name: 'Node ${{ matrix.node_cimg_tag }} unsupported'
+    runs-on: ubuntu-latest
+    needs: [build-package]
+    strategy:
+      matrix:
+        node_cimg_tag: [ 10, 12 ]
+    steps:
+      - name: Set up Node
+        uses: actions/setup-node@v3
+        with:
+          node-version: ${{ matrix.node_cimg_tag }}
+          architecture: x64
+      - name: Download distro
+        uses: actions/download-artifact@v3
+        with:
+          name: distro-package
+      - run: npm i ./distro.tgz
+      - run: |
+          output=$(node -e "require('@lumigo/opentelemetry')" 2>&1)
+
+          if [[ "${output}" != "@lumigo/opentelemetry: Node.js version '$(node --version)' is not supported"* ]]; then
+            echo "Unexpected output: ${output}"
+            exit 1
+          fi
+
+  all-tests:
+    runs-on: ubuntu-latest
+    needs:
+    - supported-node-versions-testing
+    - unsupported-node-versions-testing
+    steps:
+      - run: echo "Hurray!"

.gitignore

@@ -106,3 +106,6 @@ dist
 
 test/component/node/package-lock.json
 test/component/node/spans-test*.json
+
+# Local VSCode settings
+.vscode

.npmignore

@@ -1,4 +1,5 @@
 *
+!dist/**/**.json
 !dist/**/**.js
 !dist/**/**.js.map
 !dist/**/**.ts

README.md

@@ -232,7 +232,7 @@ In case your execution tags on different spans appear on different invocations t
 
 ## Supported runtimes
 
-* Node.js: 14.x, 16.x, 18.x
+* Node.js: 14.x, 16.x, 18.x, 20.x
 
 ## Supported packages
 

jest.instrumentations.config.js

@@ -0,0 +1,16 @@
+const allInstrumentations = '**';
+const instrumentationToTest = process.env.INSTRUMENTATION_UNDER_TEST || allInstrumentations;
+
+/** @type {import('jest').Config} */
+const config = {
+  testEnvironment: 'node',
+  testMatch: [`**/instrumentations/${instrumentationToTest}/*.test.ts`],
+  roots: ['./test'],
+  setupFilesAfterEnv: ['./jest.instrumentations.setup.js', 'jest-json', 'jest-expect-message'],
+  reporters: [
+    'default',
+    'jest-summarizing-reporter'
+  ]
+};
+
+module.exports = config;

jest.integration.setup.js → jest.instrumentations.setup.js

@@ -15,12 +15,12 @@ afterEach(() => {
 
 beforeAll(() => {
   global.console = require('console');
-  require('console-stamp')(global.console)
+  require('console-stamp')(global.console);
 });
 
 afterAll(() => {
   console.info('Starting afterAll...');
-  const versions = instrumentationsVersionManager.getInstrumantaionsVersions();
+  const versions = instrumentationsVersionManager.getInstrumantationsVersions();
   const versions_keys = Object.keys(versions);
   if (versions_keys.length) {
     versions_keys.forEach((lib) => {