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

Port testinstall to docker+go #1603

Merged
merged 2 commits into from Jan 31, 2020
Merged

Port testinstall to docker+go #1603

merged 2 commits into from Jan 31, 2020

Conversation

tadaskay
Copy link
Member

Fixes: #1545

Rewrote bash+vagrant to go+docker

@codecov-io
Copy link

codecov-io commented Jan 31, 2020

Codecov Report

Merging #1603 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1603   +/-   ##
======================================
  Coverage    47.8%   47.8%           
======================================
  Files         289     289           
  Lines       11519   11519           
======================================
  Hits         5507    5507           
  Misses       5618    5618           
  Partials      394     394
Impacted Files Coverage Δ
tequilapi/api_test_utils.go 86.66% <100%> (ø) ⬆️

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 0c874fa...7c074e5. Read the comment docs.

Copy link
Contributor

@anjmao anjmao left a comment

Choose a reason for hiding this comment

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

Great!

stretchr/testify#808 is merged into master,
but not yet released as of moment of writing. Updating to latest master version.
@tadaskay tadaskay merged commit 158f11d into master Jan 31, 2020
@tadaskay tadaskay deleted the 1545-test-install-script branch January 31, 2020 10:09
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.

Create automatic tests for install script
5 participants