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

Introduce e2e tests #174

Merged
merged 8 commits into from
Oct 15, 2018
Merged

Introduce e2e tests #174

merged 8 commits into from
Oct 15, 2018

Conversation

mavimo
Copy link
Collaborator

@mavimo mavimo commented Aug 16, 2018

Refs #122

First tentative to introduce end-to-end test, currently only check that the command are invoked correctly and do not return issue.
This branch include also the fix reported in #173 used to set token during context creation.

@xetys @JohnnyQQQQ this PR is not mergeable but is a starting point to discuss how to setup an appropriate E2E test suite

NB: PR fail since need config on travis-CI side, you can see the build working on a configured env in: https://travis-ci.org/mavimo/hetzner-kube/branches branch introduce-e2e-tests

@mavimo mavimo force-pushed the introduce-e2e-tests branch from b408f1c to a59410c Compare August 16, 2018 14:26
@xetys
Copy link
Owner

xetys commented Aug 23, 2018

That's so really amazing, as I still didn't find any time do this very important mission!

I will first take a closer look at your previous PRs before digging into this one, but that is awesome!!!

@mavimo mavimo force-pushed the introduce-e2e-tests branch from a59410c to c1dcc3d Compare August 25, 2018 11:05
@mavimo mavimo force-pushed the introduce-e2e-tests branch 7 times, most recently from 6f11b0a to dc4aada Compare September 1, 2018 16:48
@mavimo mavimo force-pushed the introduce-e2e-tests branch 9 times, most recently from d1bef8e to f30b0e2 Compare October 7, 2018 21:18
@mavimo
Copy link
Collaborator Author

mavimo commented Oct 13, 2018

@xetys the build is failing since the HETZNER_API_KEY is missing from travis config, once it is set you should se a build correctly running 😄

Eg: https://travis-ci.org/mavimo/hetzner-kube/builds/438378321

@mavimo mavimo force-pushed the introduce-e2e-tests branch from f30b0e2 to c3cfe78 Compare October 14, 2018 15:21
@mavimo mavimo force-pushed the introduce-e2e-tests branch from c3cfe78 to 4981720 Compare October 14, 2018 19:30
@xetys
Copy link
Owner

xetys commented Oct 15, 2018

we try to solve the issue with the key by merging that

@xetys xetys merged commit 8012f31 into xetys:master Oct 15, 2018
@mavimo mavimo deleted the introduce-e2e-tests branch October 15, 2018 14:47
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.

2 participants