Skip to content

chore(todo): add missing span sampling test#1151

Merged
mayurkale22 merged 4 commits intoopen-telemetry:masterfrom
markwolff:add-missing-span-recording-test
Jun 9, 2020
Merged

chore(todo): add missing span sampling test#1151
mayurkale22 merged 4 commits intoopen-telemetry:masterfrom
markwolff:add-missing-span-recording-test

Conversation

@markwolff
Copy link
Member

Short description of the changes

  • Cleanup todo in tests

@codecov
Copy link

codecov bot commented Jun 9, 2020

Codecov Report

Merging #1151 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1151   +/-   ##
=======================================
  Coverage   92.31%   92.31%           
=======================================
  Files         116      116           
  Lines        3395     3395           
  Branches      688      688           
=======================================
  Hits         3134     3134           
  Misses        261      261           

@dyladan dyladan added the Merge:LGTM This PR is ready to be merged by a Maintainer (has enough valid approvals, successful build, etc.) label Jun 9, 2020
@mayurkale22 mayurkale22 merged commit 1af5928 into open-telemetry:master Jun 9, 2020
@mayurkale22 mayurkale22 added the enhancement New feature or request label Jun 9, 2020
pichlermarc pushed a commit to dynatrace-oss-contrib/opentelemetry-js that referenced this pull request Dec 15, 2023
…lemetry#1151)

* Add tests to validate that requireParentSpan applies to connect

* Copy finishedSpans

* Implement requireParentSpan for connect

Co-authored-by: Amir Blum <amir@aspecto.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Merge:LGTM This PR is ready to be merged by a Maintainer (has enough valid approvals, successful build, etc.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants