[test optimization] Fix happy-dom testing issues with latest jest release (#6566)

juan-fernandez · web-flow · commit 5374f69203c4 · 2025-10-02T10:42:13.000+02:00
diff --git a/integration-tests/jest/jest.spec.js b/integration-tests/jest/jest.spec.js
@@ -62,6 +62,7 @@ const {
 } = require('../../packages/dd-trace/src/plugins/util/test')
 const { DD_HOST_CPU_COUNT } = require('../../packages/dd-trace/src/plugins/util/env')
 const { ERROR_MESSAGE } = require('../../packages/dd-trace/src/constants')
+const { NODE_MAJOR } = require('../../version')
 
 const testFile = 'ci-visibility/run-jest.js'
 const expectedStdout = 'Test Suites: 2 passed'
@@ -2286,8 +2287,9 @@ describe('jest CommonJS', () => {
         }).catch(done)
       })
     })
-
-    it('works with happy-dom', async () => {
+    // happy-dom>=19 can only be used with CJS from node 20 and above
+    const happyDomTest = NODE_MAJOR < 20 ? it.skip : it
+    happyDomTest('works with happy-dom', async () => {
       // Tests from ci-visibility/test/ci-visibility-test-2.js will be considered new
       receiver.setKnownTests({
         jest: {
diff --git a/packages/datadog-plugin-jest/test/circus.spec.js b/packages/datadog-plugin-jest/test/circus.spec.js
@@ -48,7 +48,6 @@ function loadAgent (moduleName, version, isAgentlessTest, isEvpProxyTest) {
   if (!isEvpProxyTest) {
     agent.setAvailableEndpoints([])
   }
-  const isHappyDom = moduleName === '@happy-dom/jest-environment'
   return agent.load(
     ['jest', 'http'],
     { service: 'test' },
@@ -58,19 +57,15 @@ function loadAgent (moduleName, version, isAgentlessTest, isEvpProxyTest) {
       global.__libraryVersion__ = version
 
       return {
-        jestExecutable: isHappyDom
-          ? require('../../../versions/jest').get()
-          : require(`../../../versions/jest@${version}`).get(),
+        jestExecutable: require(`../../../versions/jest@${version}`).get(),
         jestCommonOptions: {
           projects: [__dirname],
           testPathIgnorePatterns: ['/node_modules/'],
           coverageReporters: ['none'],
           reporters: [],
           silent: true,
-          testEnvironment: isHappyDom ? '@happy-dom/jest-environment' : path.join(__dirname, 'env.js'),
-          testRunner: isHappyDom
-            ? require('../../../versions/jest-circus').getPath('jest-circus/runner')
-            : require(`../../../versions/jest-circus@${version}`).getPath('jest-circus/runner'),
+          testEnvironment: path.join(__dirname, 'env.js'),
+          testRunner: require(`../../../versions/jest-circus@${version}`).getPath('jest-circus/runner'),
           cache: false,
           maxWorkers: '50%'
         }
@@ -85,7 +80,7 @@ describe('Plugin', function () {
   this.timeout(testTimeout)
   this.retries(2)
 
-  const versions = ['jest-environment-node', 'jest-environment-jsdom', '@happy-dom/jest-environment']
+  const versions = ['jest-environment-node', 'jest-environment-jsdom']
 
   withVersions('jest', versions, (version, moduleName) => {
     afterEach(() => {
