Skip to content

Comments

Auto-activate nodejs manifest tests#6378

Draft
dd-octo-sts[bot] wants to merge 1 commit intomainfrom
auto-nodejs-manifest
Draft

Auto-activate nodejs manifest tests#6378
dd-octo-sts[bot] wants to merge 1 commit intomainfrom
auto-nodejs-manifest

Conversation

@dd-octo-sts
Copy link
Contributor

@dd-octo-sts dd-octo-sts bot commented Feb 25, 2026

Automated activation of easy-win tests for nodejs
View nightly workflow run

  • If you approve this PR please also merge it.
  • If the tests are failing it might be due to a change made since the last nightly system-tests run. You can close the PR, an updated one will be available tomorrow.
  • If you close the PR please also delete the branch

This commit was created from the local commit with hash b13285fbbf01f96e437ff957c85f74c96cd8be1a.
@dd-octo-sts dd-octo-sts bot requested a review from a team as a code owner February 25, 2026 09:05
@dd-octo-sts dd-octo-sts bot enabled auto-merge (squash) February 25, 2026 09:05
@github-actions
Copy link
Contributor

CODEOWNERS have been resolved as:

manifests/nodejs.yml                                                    @DataDog/dd-trace-js

@datadog-datadog-prod-us1
Copy link

datadog-datadog-prod-us1 bot commented Feb 25, 2026

⚠️ Tests

Fix all issues with BitsAI or with Cursor

⚠️ Warnings

❄️ 2 New flaky tests detected

    tests.stats.test_stats.Test_Time_Bucketing.test_client_side_stats[fastify] from system_tests_suite (Datadog) (Fix with Cursor)

    tests.stats.test_stats.Test_Time_Bucketing.test_client_side_stats_bucket_alignment[fastify] from system_tests_suite (Datadog) (Fix with Cursor)

🧪 2 Tests failed

tests.test_telemetry.Test_DependencyEnable.test_app_dependency_loaded_not_sent_dependency_collection_disabled[express4] from system_tests_suite (Datadog) (Fix with Cursor)
ValueError: request_type app-dependencies-loaded should not be sent by this tracer

self = <tests.test_telemetry.Test_DependencyEnable object at 0x7fea19aa7740>

    def test_app_dependency_loaded_not_sent_dependency_collection_disabled(self):
        """app-dependencies-loaded request should not be sent if DD_TELEMETRY_DEPENDENCY_COLLECTION_ENABLED is false"""
    
        for data in interfaces.library.get_telemetry_data():
            if get_request_type(data) == "app-dependencies-loaded":
>               raise ValueError("request_type app-dependencies-loaded should not be sent by this tracer")
...
tests.test_telemetry.Test_DependencyEnable.test_app_dependency_loaded_not_sent_dependency_collection_disabled[uds-express4] from system_tests_suite (Datadog) (Fix with Cursor)
ValueError: request_type app-dependencies-loaded should not be sent by this tracer

self = <tests.test_telemetry.Test_DependencyEnable object at 0x7fa2c4f21850>

    def test_app_dependency_loaded_not_sent_dependency_collection_disabled(self):
        """app-dependencies-loaded request should not be sent if DD_TELEMETRY_DEPENDENCY_COLLECTION_ENABLED is false"""
    
        for data in interfaces.library.get_telemetry_data():
            if get_request_type(data) == "app-dependencies-loaded":
>               raise ValueError("request_type app-dependencies-loaded should not be sent by this tracer")
...
This comment will be updated automatically if new data arrives.
🔗 Commit SHA: b947069 | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@nccatoni nccatoni marked this pull request as draft February 25, 2026 12:50
auto-merge was automatically disabled February 25, 2026 12:50

Pull request was converted to draft

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants