Skip to content

Move flag.Parse in tests to TestMain#3744

Merged
estesp merged 1 commit intocontainerd:masterfrom
crosbymichael:test-flags
Oct 11, 2019
Merged

Move flag.Parse in tests to TestMain#3744
estesp merged 1 commit intocontainerd:masterfrom
crosbymichael:test-flags

Conversation

@crosbymichael
Copy link
Copy Markdown
Member

This this fixes issues with custom and testing flags in Go 1.3 and should work
in previous go versions.

Signed-off-by: Michael Crosby [email protected]

This this fixes issues with custom and testing flags in Go 1.3 and should work
in previous go versions.

Signed-off-by: Michael Crosby <[email protected]>
Copy link
Copy Markdown
Member

@cpuguy83 cpuguy83 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Oct 11, 2019

Build succeeded.

@codecov-io
Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3744      +/-   ##
==========================================
+ Coverage   41.98%   45.39%   +3.41%     
==========================================
  Files         131      118      -13     
  Lines       14528    11639    -2889     
==========================================
- Hits         6099     5284     -815     
+ Misses       7521     5448    -2073     
+ Partials      908      907       -1
Flag Coverage Δ
#linux 45.39% <ø> (ø) ⬆️
#windows ?
Impacted Files Coverage Δ
remotes/docker/fetcher.go 42.5% <0%> (-7.5%) ⬇️
remotes/docker/auth.go 63.82% <0%> (-3.97%) ⬇️
remotes/docker/status.go 21.42% <0%> (-3.58%) ⬇️
remotes/docker/errdesc.go 28.12% <0%> (-2.65%) ⬇️
remotes/docker/errcode.go 47.61% <0%> (-1.6%) ⬇️
platforms/cpuinfo.go 3.77% <0%> (-0.85%) ⬇️
log/context.go 36.84% <0%> (-0.66%) ⬇️
remotes/docker/handler.go 22.38% <0%> (-0.09%) ⬇️
errdefs/grpc.go 77.46% <0%> (-0.04%) ⬇️
remotes/docker/pusher.go 0% <0%> (ø) ⬆️
... and 75 more

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 38937f0...d5b7bf5. Read the comment docs.

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.

4 participants