Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: fixing failing test #1747

Merged
merged 2 commits into from
Dec 14, 2020
Merged

feat: fixing failing test #1747

merged 2 commits into from
Dec 14, 2020

Conversation

obecny
Copy link
Member

@obecny obecny commented Dec 14, 2020

Fixes #1745

Update:
the 2nd link (info.cern.ch) used in test is not valid anymore
http instrumentation and http plugin was using the same port

@codecov
Copy link

codecov bot commented Dec 14, 2020

Codecov Report

Merging #1747 (6d9bca9) into master (5701473) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1747      +/-   ##
==========================================
- Coverage   92.18%   92.17%   -0.02%     
==========================================
  Files         167      167              
  Lines        5594     5594              
  Branches     1191     1191              
==========================================
- Hits         5157     5156       -1     
- Misses        437      438       +1     
Impacted Files Coverage Δ
...emetry-core/src/platform/node/RandomIdGenerator.ts 87.50% <0.00%> (-6.25%) ⬇️

@obecny
Copy link
Member Author

obecny commented Dec 14, 2020

@open-telemetry/javascript-approvers need approves to unblock other PRs

Copy link
Member

@dyladan dyladan left a comment

Choose a reason for hiding this comment

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

Seems fine but can you provide a short explanation why this fixes the flaky test?

@obecny
Copy link
Member Author

obecny commented Dec 14, 2020

Seems fine but can you provide a short explanation why this fixes the flaky test?

updated description

@obecny obecny merged commit d113742 into open-telemetry:master Dec 14, 2020
@obecny obecny deleted the http-test branch December 14, 2020 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix failing test for http plugin
3 participants