Skip to content

Add unit tests for SelfDiagnosticsModule#1706

Merged
cijothomas merged 6 commits intoopen-telemetry:mainfrom
xiang17:xiang17/UnitTestsForSelfDiagnosticsConfigRefresher
Jan 29, 2021
Merged

Add unit tests for SelfDiagnosticsModule#1706
cijothomas merged 6 commits intoopen-telemetry:mainfrom
xiang17:xiang17/UnitTestsForSelfDiagnosticsConfigRefresher

Conversation

@xiang17
Copy link
Copy Markdown
Contributor

@xiang17 xiang17 commented Jan 19, 2021

Add unit tests that validate the fixes for #1689 works. The tests should pass once PRs #1690 and #1693 are merged.

Changes

Add unit tests for SelfDiagnosticsModule

@xiang17 xiang17 requested a review from a team January 19, 2021 21:14
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 22, 2021

Codecov Report

Merging #1706 (5a2be11) into main (ac6689c) will increase coverage by 0.26%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1706      +/-   ##
==========================================
+ Coverage   82.89%   83.16%   +0.26%     
==========================================
  Files         193      193              
  Lines        6005     6005              
==========================================
+ Hits         4978     4994      +16     
+ Misses       1027     1011      -16     
Impacted Files Coverage Δ
...lemetry/Internal/SelfDiagnosticsConfigRefresher.cs 89.62% <0.00%> (+15.09%) ⬆️

@eddynaka eddynaka changed the base branch from master to main January 27, 2021 22:14
xiang17 and others added 2 commits January 29, 2021 14:20
…resher

# Conflicts:
#	test/OpenTelemetry.Tests/Internal/SelfDiagnosticsConfigRefresherTest.cs
@cijothomas cijothomas merged commit 416a6c1 into open-telemetry:main Jan 29, 2021
@xiang17 xiang17 deleted the xiang17/UnitTestsForSelfDiagnosticsConfigRefresher branch January 29, 2021 23:28
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.

2 participants