Skip to content

Improve test coverage to ~100% for @azure/core-tracing#38181

Merged
deyaaeldeen merged 1 commit intomainfrom
deyaaeldeen/core-tracing-test-coverage
Apr 17, 2026
Merged

Improve test coverage to ~100% for @azure/core-tracing#38181
deyaaeldeen merged 1 commit intomainfrom
deyaaeldeen/core-tracing-test-coverage

Conversation

@deyaaeldeen
Copy link
Copy Markdown
Member

@deyaaeldeen deyaaeldeen commented Apr 16, 2026

Adds tests to bring @azure/core-tracing to ~100% test coverage.

Changes

  • Extended tracingClient tests with edge cases
  • 1 file changed, 25 lines added

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@deyaaeldeen deyaaeldeen marked this pull request as ready for review April 17, 2026 00:18
@deyaaeldeen deyaaeldeen requested review from a team and maorleger as code owners April 17, 2026 00:18
Copilot AI review requested due to automatic review settings April 17, 2026 00:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves test coverage for @azure/core-tracing by adding targeted unit tests for TracingClient methods that delegate to the configured instrumenter, helping close remaining uncovered lines without changing runtime behavior.

Changes:

  • Added a new #parseTraceparentHeader test verifying delegation to the instrumenter and return value passthrough.
  • Added a new #createRequestHeaders test verifying delegation to the instrumenter and return value passthrough.

@deyaaeldeen deyaaeldeen merged commit 64482d8 into main Apr 17, 2026
20 checks passed
@deyaaeldeen deyaaeldeen deleted the deyaaeldeen/core-tracing-test-coverage branch April 17, 2026 21:39
deyaaeldeen added a commit that referenced this pull request Apr 18, 2026
Adds tests to bring `@azure/core-tracing` to ~100% test coverage.


### Changes
- Extended tracingClient tests with edge cases
- 1 file changed, 25 lines added

Co-authored-by: Deyaaeldeen Almahallawi <deyaa@microsoft.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants