Skip to content
This repository was archived by the owner on Jul 18, 2025. It is now read-only.

Refactor e2e tests to use "gotest.tools/fs" dir#459

Merged
silvin-lubecki merged 1 commit intodocker-archive-public:masterfrom
ulyssessouza:refactor-e2e-tmp-fs
Feb 12, 2019
Merged

Refactor e2e tests to use "gotest.tools/fs" dir#459
silvin-lubecki merged 1 commit intodocker-archive-public:masterfrom
ulyssessouza:refactor-e2e-tmp-fs

Conversation

@ulyssessouza
Copy link
Copy Markdown
Contributor

- What I did
Refactor tests not to pollute the fs during the e2e execution

- How I did it
Refactor e2e tests to use "gotest.tools/fs" created dir instead of current folder

- How to verify it
By monitoring the creation of files with something like inotifywait -r -m e2e

- Description for the changelog
Refactor e2e tests to use "gotest.tools/fs" dir

- A picture of a cute animal (not mandatory but encouraged)
monkey

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 8, 2019

Codecov Report

Merging #459 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #459   +/-   ##
=======================================
  Coverage   66.42%   66.42%           
=======================================
  Files          51       51           
  Lines        2740     2740           
=======================================
  Hits         1820     1820           
  Misses        660      660           
  Partials      260      260

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 ec9d985...5c667be. Read the comment docs.

Copy link
Copy Markdown
Contributor

@simonferquel simonferquel left a comment

Choose a reason for hiding this comment

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

lgtm

Comment thread e2e/commands_test.go Outdated
Copy link
Copy Markdown
Contributor

@chris-crone chris-crone left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread e2e/commands_test.go Outdated
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
@ulyssessouza ulyssessouza changed the title WIP - Refactor e2e tests to use "gotest.tools/fs" dir Refactor e2e tests to use "gotest.tools/fs" dir Feb 12, 2019
@ulyssessouza
Copy link
Copy Markdown
Contributor Author

@rumpl @silvin-lubecki PTAL

Copy link
Copy Markdown
Contributor

@silvin-lubecki silvin-lubecki left a comment

Choose a reason for hiding this comment

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

LGTM

@silvin-lubecki silvin-lubecki merged commit 0c1ee92 into docker-archive-public:master Feb 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants