Skip to content

Add TracerProvider tests to oteltest harness#1607

Merged
MrAlias merged 7 commits intoopen-telemetry:mainfrom
MrAlias:test-tp-concur
Mar 3, 2021
Merged

Add TracerProvider tests to oteltest harness#1607
MrAlias merged 7 commits intoopen-telemetry:mainfrom
MrAlias:test-tp-concur

Conversation

@MrAlias
Copy link
Copy Markdown
Contributor

@MrAlias MrAlias commented Feb 26, 2021

Related to #1587

Ensure our TracerProvider.Tracer method is safe to call concurrently and that multiple TracerProviders can be created as required by the specification.

@MrAlias MrAlias added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Feb 26, 2021
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 26, 2021

Codecov Report

Merging #1607 (46369cc) into main (bb4c297) will decrease coverage by 0.3%.
The diff coverage is 0.0%.

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #1607     +/-   ##
=======================================
- Coverage   77.6%   77.2%   -0.4%     
=======================================
  Files        128     128             
  Lines       6637    6669     +32     
=======================================
  Hits        5155    5155             
- Misses      1235    1267     +32     
  Partials     247     247             
Impacted Files Coverage Δ
internal/matchers/expectation.go 0.0% <0.0%> (ø)
oteltest/harness.go 82.7% <0.0%> (-17.3%) ⬇️
sdk/trace/provider.go 95.0% <ø> (ø)
trace/trace.go 86.6% <ø> (ø)

@MrAlias MrAlias added the pkg:testing Related to testing or a testing package label Mar 3, 2021
@MrAlias MrAlias merged commit 992837f into open-telemetry:main Mar 3, 2021
@MrAlias MrAlias deleted the test-tp-concur branch March 3, 2021 20:10
ldelossa pushed a commit to ldelossa/opentelemetry-go that referenced this pull request Mar 5, 2021
* Add TracerProvider tests to oteltest harness

* Update Tracer method docs

* Fix grammar
@pellared pellared added this to the untracked milestone Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg:testing Related to testing or a testing package Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants