test: migrate tap to mocha

[BridgeAR]

This tries to align with the former behavior as close as possible. This is for example the case by using more parallel threads, even if the CPU has less cores.

[github-actions]

Overall package size

Self size: 4.37 MB
Deduped: 5.19 MB
No deduping: 5.19 MB

Dependency sizes

| name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 2.0.0 | 68.46 kB | 797.03 kB | | dc-polyfill | 0.1.10 | 26.73 kB | 26.73 kB |

_{🤖 This report was automatically generated by heaviest-objects-in-the-universe}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.51%. Comparing base (28b054c) to head (75b81bd).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #7135      +/-   ##
==========================================
- Coverage   84.53%   84.51%   -0.02%     
==========================================
  Files         526      527       +1     
  Lines       22534    22538       +4     
==========================================
  Hits        19048    19048              
- Misses       3486     3490       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[pr-commenter]

Benchmarks

Benchmark execution time: 2026-01-07 18:33:51

Comparing candidate commit 75b81bd in PR branch BridgeAR/2025-12-18-migrate-tap-to-mocha with baseline commit 28b054c in branch master.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 292 metrics, 28 unstable metrics.

[datadog-datadog-prod-us1]

⚠️ Tests

✨ Fix all issues with Cursor

⚠️ Warnings

❄️ 4 New flaky tests detected

tests.integrations.crossed_integrations.test_rabbitmq.Test_RabbitMQ_Trace_Context_Propagation.test_consume[fastify] from system_tests_suite (Datadog) (Fix with Cursor)

assert None is not None

self = <tests.integrations.crossed_integrations.test_rabbitmq.Test_RabbitMQ_Trace_Context_Propagation object at 0x7fcbf2dda780>

    def test_consume(self):
        """Check that a message by an app instrumented by a Datadog tracer is correctly ingested"""
    
        assert self.production_response.status_code == 200, self.production_response.text
        assert self.consume_response.status_code == 200, self.consume_response.text
    
...

tests.integrations.crossed_integrations.test_rabbitmq.Test_RabbitMQ_Trace_Context_Propagation.test_consume_trace_equality[fastify] from system_tests_suite (Datadog) (Fix with Cursor)

assert None is not None

self = <tests.integrations.crossed_integrations.test_rabbitmq.Test_RabbitMQ_Trace_Context_Propagation object at 0x7fcbf2dda210>

    @missing_feature(library="golang", reason="Expected to fail, Golang does not propagate context")
    @missing_feature(library="ruby", reason="Expected to fail, Ruby does not propagate context")
    def test_consume_trace_equality(self):
        """This test relies on the setup for consume, it currently cannot be run on its own"""
        producer_span = self.get_span(
            self.buddy_interface,
...

tests.integrations.crossed_integrations.test_rabbitmq.Test_RabbitMQ_Trace_Context_Propagation.test_produce[fastify] from system_tests_suite (Datadog) (Fix with Cursor)

assert None is not None

self = <tests.integrations.crossed_integrations.test_rabbitmq.Test_RabbitMQ_Trace_Context_Propagation object at 0x7fcbf2ddafc0>

    def test_produce(self):
        """Check that a message produced to RabbitMQ is correctly ingested by a Datadog tracer"""
    
        assert self.production_response.status_code == 200, self.production_response.text
        assert self.consume_response.status_code == 200, self.consume_response.text
    
...

tests.integrations.crossed_integrations.test_rabbitmq.Test_RabbitMQ_Trace_Context_Propagation.test_produce_trace_equality[fastify] from system_tests_suite (Datadog) (Fix with Cursor)

assert None is not None

self = <tests.integrations.crossed_integrations.test_rabbitmq.Test_RabbitMQ_Trace_Context_Propagation object at 0x7fcbf2ddb3e0>

    @missing_feature(library="golang", reason="Expected to fail, Golang does not propagate context")
    @missing_feature(library="ruby", reason="Expected to fail, Ruby does not propagate context")
    def test_produce_trace_equality(self):
        """This test relies on the setup for produce, it currently cannot be run on its own"""
        producer_span = self.get_span(
            interfaces.library,
...

View all

ℹ️ Info

🧪 All tests passed

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 75b81bd | Docs | Datadog PR Page | Was this helpful? Give us feedback!}

[tlhunter]

As mentioned elsewhere my preference for test suites is:
all node:test > all tap > all mocha > tap + mocha

So I would consider this PR an improvement to what we have today even if it's not in the direction I would ultimately prefer.

[watson]

Forgot to mention in my review that this PR should also update the .md files currently mentioning tap

[watson]

I've reviewed everything except the new scripts in scripts folder for running the mocha tests in parallel. I don't have the bandwith to review those now, but I think as long as we can verify that all the tests are running and that if a single test is failing, it fails the job, then I'm happy with this getting merged.