Skip to content

Try to stabilize tests - #1285

Merged
bsideup merged 9 commits into
masterfrom
stabilize_tests
Dec 9, 2019
Merged

bsideup merged 9 commits into
masterfrom
stabilize_tests

Conversation

@bsideup

@bsideup bsideup commented Dec 7, 2019

Copy link
Copy Markdown
Member
  • perform cleanup after each test
  • improve AttachContainerCmdIT

This change is Reviewable

- perform cleanup after each test
- improve AttachContainerCmdIT
@codecov-io

codecov-io commented Dec 9, 2019

Copy link
Copy Markdown

Codecov Report

Merging #1285 into master will increase coverage by 18.75%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master    #1285       +/-   ##
===========================================
+ Coverage      75%   93.75%   +18.75%     
===========================================
  Files           1        1               
  Lines          16       16               
===========================================
+ Hits           12       15        +3     
+ Misses          3        1        -2     
+ Partials        1        0        -1
Impacted Files Coverage Δ
...om/github/dockerjava/core/DockerClientBuilder.java 93.75% <0%> (+18.75%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 217c14b...98fe92c. Read the comment docs.

@bsideup
bsideup merged commit 93202d1 into master Dec 9, 2019
@bsideup
bsideup deleted the stabilize_tests branch December 9, 2019 15:27
@bsideup bsideup mentioned this pull request Dec 9, 2019
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