[ci-visibility] Use system's node instead of the bundled one in cypre…

…ss tests (#4011)

.github/workflows/project.yml

@@ -53,6 +53,9 @@ jobs:
   integration-cypress:
     strategy:
       matrix:
+        # Important: This is outside the minimum supported version of dd-trace-js
+        # Node > 16 does not work with [email protected] (not even without our plugin)
+        # TODO: figure out what to do with this: we might have to deprecate support for [email protected]
         version: [16, latest]
         # 6.7.0 is the minimum version we support
         cypress-version: [6.7.0, latest]

integration-tests/cypress-config.json

@@ -4,5 +4,6 @@
   "pluginsFile": "cypress/plugins-old/index.js",
   "supportFile": "cypress/support/e2e.js",
   "integrationFolder": "cypress/e2e",
-  "defaultCommandTimeout": 100
+  "defaultCommandTimeout": 100,
+  "nodeVersion": "system"
 }

integration-tests/cypress/cypress.spec.js

@@ -30,6 +30,7 @@ const {
 } = require('../../packages/dd-trace/src/plugins/util/test')
 const { ERROR_MESSAGE } = require('../../packages/dd-trace/src/constants')
 const semver = require('semver')
+const { NODE_MAJOR } = require('../../version')
 
 const version = process.env.CYPRESS_VERSION
 const hookFile = 'dd-trace/loader-hook.mjs'
@@ -56,6 +57,9 @@ moduleType.forEach(({
   if (type === 'esm' && semver.satisfies(version, '<10.0.0')) {
     return
   }
+  if (version === '6.7.0' && NODE_MAJOR > 16) {
+    return
+  }
   describe(`cypress@${version} ${type}`, function () {
     this.retries(2)
     this.timeout(60000)