Skip to content
This repository has been archived by the owner on Jun 7, 2021. It is now read-only.

First test e2e #19

Merged
merged 4 commits into from
Sep 4, 2019
Merged

First test e2e #19

merged 4 commits into from
Sep 4, 2019

Conversation

slinkydeveloper
Copy link
Contributor

@slinkydeveloper slinkydeveloper commented Sep 2, 2019

Fixes #17

Created first e2e with basic support functions. It should be a good starting point for next e2e

Copy link
Member

@lance lance left a comment

Choose a reason for hiding this comment

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

Other than a couple of minor comments, this LGTM. I'm curious - do you think this could be run in a CI/CD pipeline? If so, how?

test/util.go Show resolved Hide resolved
test/util.go Show resolved Hide resolved
test/run_e2e.sh Outdated Show resolved Hide resolved
@lance
Copy link
Member

lance commented Sep 4, 2019

Please squash your commits into one.

@slinkydeveloper
Copy link
Contributor Author

I'm not sure it can run in our pipeline, since we need an openshift cluster to connect to. Honestly I don't know

@zroubalik
Copy link
Contributor

Other than a couple of minor comments, this LGTM. I'm curious - do you think this could be run in a CI/CD pipeline? If so, how?

wrt CI/CD - We might want to integrate with OpenShift CI used by Knative projects. It creates real cluster and run the tests on it. Might be exaggerated ATM. I've created an issue for this: #21

Copy link
Contributor

@zroubalik zroubalik left a comment

Choose a reason for hiding this comment

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

LGTM

@lance lance merged commit d29cdf3 into master Sep 4, 2019
@lance lance deleted the first-test-e2e branch September 4, 2019 19:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

E2E tests
3 participants