Skip to content

Comments

Auto-activate nodejs manifest tests#6373

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

Auto-activate nodejs manifest tests#6373
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 24, 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 446a6c3c5efc74585a9263781697790604759316.
@dd-octo-sts dd-octo-sts bot requested a review from a team as a code owner February 24, 2026 14:02
@dd-octo-sts dd-octo-sts bot enabled auto-merge (squash) February 24, 2026 14:02
@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 24, 2026

⚠️ Tests

Fix all issues with BitsAI or with Cursor

⚠️ Warnings

🧪 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 0x7f6a17ec4e60>

    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 0x7fd79d620dd0>

    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")
...

ℹ️ Info

❄️ No new flaky tests detected

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

@nccatoni nccatoni marked this pull request as draft February 24, 2026 14:21
auto-merge was automatically disabled February 24, 2026 14:21

Pull request was converted to draft

@nccatoni nccatoni closed this Feb 25, 2026
@nccatoni nccatoni deleted the auto-nodejs-manifest branch February 25, 2026 09:02
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.

1 participant