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

remove ignored tests in SVContextUnitTest #5159

Merged
merged 2 commits into from
Sep 17, 2018

Conversation

SHuang-Broad
Copy link
Contributor

No description provided.

@SHuang-Broad SHuang-Broad requested a review from vruano September 5, 2018 18:51
@SHuang-Broad SHuang-Broad force-pushed the sh-sv-remove-svcontes-ignored-tests branch from 017624a to e5c659e Compare September 6, 2018 02:40
@codecov-io
Copy link

Codecov Report

Merging #5159 into master will increase coverage by 0.001%.
The diff coverage is 80%.

@@               Coverage Diff               @@
##              master     #5159       +/-   ##
===============================================
+ Coverage     86.743%   86.744%   +0.001%     
- Complexity     29470     29475        +5     
===============================================
  Files           1818      1818               
  Lines         136423    136433       +10     
  Branches       15124     15126        +2     
===============================================
+ Hits          118337    118347       +10     
+ Misses         12646     12644        -2     
- Partials        5440      5442        +2
Impacted Files Coverage Δ Complexity Δ
...bender/tools/spark/sv/utils/SVContextUnitTest.java 90.769% <80%> (-2.564%) 34 <6> (+5)
...utils/smithwaterman/SmithWatermanIntelAligner.java 90% <0%> (+10%) 3% <0%> (ø) ⬇️
...ithwaterman/SmithWatermanIntelAlignerUnitTest.java 90% <0%> (+30%) 2% <0%> (ø) ⬇️

@SHuang-Broad SHuang-Broad force-pushed the sh-sv-remove-svcontes-ignored-tests branch from e5c659e to d770748 Compare September 10, 2018 22:51
@vruano
Copy link
Contributor

vruano commented Sep 17, 2018

Seems alright, but it has some conflicts now; trying to solve it via web.

@vruano vruano merged commit f81f3ed into master Sep 17, 2018
@vruano vruano deleted the sh-sv-remove-svcontes-ignored-tests branch September 17, 2018 19:38
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