Skip to content

chore(node-core): Fix node-core integration test assertions #19220

@github-actions

Description

@github-actions

Note

The pull request "chore(node-core): Fix node-core integration test assertions" was created by @andreiborza but did not reference an issue. Therefore this issue was created for better visibility in external tools like Linear.

We excluded running the node-core integration tests from ci here, but we never ensured they were ran elsewhere.

This PR fixes the failing tests and adds a dedicated step in build.yml.

As for impact: The failing tests were mostly centered around asserting the correct sdk name, and two failing tests around usage of top-level await and one around adding headers to outgoing http requests.

None of these are critical, the last one working when scoped to node 22 (expected, since our http integration on node-core uses diagnostic channels that are only available on node 22). This is not an issue in our node sdk because we use otel's http integration for that.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions