[test-optimization] [SDTEST-303] Fix jest@30 release (#4581)

Author: juan-fernandez

.github/workflows/test-optimization.yml

@@ -63,7 +63,7 @@ jobs:
     strategy:
       matrix:
         version: [oldest, latest]
-        framework: [cucumber, selenium, mocha]
+        framework: [cucumber, selenium, jest, mocha]
     runs-on: ubuntu-latest
     env:
       DD_SERVICE: dd-trace-js-integration-tests
@@ -173,21 +173,21 @@ jobs:
       - uses: codecov/codecov-action@ad3126e916f78f00edff4ed0317cf185271ccc2d # v5.4.2
 
   # TODO: fix performance issues and test more Node versions
-  # plugin-jest:
-  #   runs-on: ubuntu-latest
-  #   env:
-  #     PLUGINS: jest
-  #   steps:
-  #     - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
-  #     - uses: ./.github/actions/testagent/start
-  #     - uses: ./.github/actions/node/active-lts
-  #     - uses: ./.github/actions/install
-  #     - run: yarn test:plugins:ci
-  #     - if: always()
-  #       uses: ./.github/actions/testagent/logs
-  #       with:
-  #         suffix: plugins-${{ github.job }}
-  #     - uses: codecov/codecov-action@ad3126e916f78f00edff4ed0317cf185271ccc2d # v5.4.2
+  plugin-jest:
+    runs-on: ubuntu-latest
+    env:
+      PLUGINS: jest
+    steps:
+      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
+      - uses: ./.github/actions/testagent/start
+      - uses: ./.github/actions/node/active-lts
+      - uses: ./.github/actions/install
+      - run: yarn test:plugins:ci
+      - if: always()
+        uses: ./.github/actions/testagent/logs
+        with:
+          suffix: plugins-${{ github.job }}
+      - uses: codecov/codecov-action@ad3126e916f78f00edff4ed0317cf185271ccc2d # v5.4.2
 
   plugin-mocha:
     runs-on: ubuntu-latest

integration-tests/ci-visibility/jest-custom-test-sequencer.js

@@ -7,8 +7,7 @@ const options = {
   testRegex: process.env.TESTS_TO_RUN ? new RegExp(process.env.TESTS_TO_RUN) : /test\/ci-visibility-test/,
   coverage: !process.env.DISABLE_CODE_COVERAGE,
   runInBand: true,
-  shard: process.env.TEST_SHARD || undefined,
-  testSequencer: process.env.CUSTOM_TEST_SEQUENCER
+  shard: process.env.TEST_SHARD || undefined
 }
 
 if (process.env.RUN_IN_PARALLEL) {

integration-tests/ci-visibility/run-jest.js

@@ -1427,61 +1427,6 @@ describe('jest CommonJS', () => {
       })
     })
 
-    it('can skip when using a custom test sequencer', (done) => {
-      receiver.setSettings({
-        itr_enabled: true,
-        tests_skipping: true
-      })
-      receiver.setSuitesToSkip([{
-        type: 'suite',
-        attributes: {
-          suite: 'ci-visibility/test/ci-visibility-test.js'
-        }
-      }])
-
-      const eventsPromise = receiver
-        .gatherPayloadsMaxTimeout(({ url }) => url.endsWith('/api/v2/citestcycle'), (payloads) => {
-          const events = payloads.flatMap(({ payload }) => payload.events)
-          const testEvents = events.filter(event => event.type === 'test')
-          // no tests end up running (suite is skipped)
-          assert.equal(testEvents.length, 0)
-
-          const testSession = events.find(event => event.type === 'test_session_end').content
-          assert.propertyVal(testSession.meta, TEST_ITR_TESTS_SKIPPED, 'true')
-
-          const skippedSuite = events.find(event =>
-            event.content.resource === 'test_suite.ci-visibility/test/ci-visibility-test.js'
-          ).content
-          assert.propertyVal(skippedSuite.meta, TEST_STATUS, 'skip')
-          assert.propertyVal(skippedSuite.meta, TEST_SKIPPED_BY_ITR, 'true')
-        })
-      childProcess = exec(
-        runTestsWithCoverageCommand,
-        {
-          cwd,
-          env: {
-            ...getCiVisAgentlessConfig(receiver.port),
-            CUSTOM_TEST_SEQUENCER: './ci-visibility/jest-custom-test-sequencer.js',
-            TEST_SHARD: '2/2'
-          },
-          stdio: 'inherit'
-        }
-      )
-      childProcess.stdout.on('data', (chunk) => {
-        testOutput += chunk.toString()
-      })
-      childProcess.stderr.on('data', (chunk) => {
-        testOutput += chunk.toString()
-      })
-
-      childProcess.on('exit', () => {
-        assert.include(testOutput, 'Running shard with a custom sequencer')
-        eventsPromise.then(() => {
-          done()
-        }).catch(done)
-      })
-    })
-
     it('works with multi project setup and test skipping', (done) => {
       receiver.setSettings({
         itr_enabled: true,