giantswarm / endpoint-operator

Apache License 2.0
2 stars 1 forks source link

Add e2e tests #18

Open MarcelMue opened 6 years ago

MarcelMue commented 6 years ago

This is mainly for cases where unit tests with a fake k8s client don't work properly. Description by @fgimenez :

Also not related to this PR, could it be possible to add some e2e tests here? For reference, with these in place, we could install the endpoint-operator-chart built from the branch and test it on a k8s cluster, either with out-of-cluster (for example https://github.com/giantswarm/aws-operator/blob/master/e2e/project.yaml#L33) or in-cluster tests (like https://github.com/giantswarm/aws-operator/blob/master/e2e/tests/awstpr.go#L34). Let me know if I can be of any help with that.

fgimenez commented 6 years ago

It would involve these:

I can propose a PR with a basic skeleton so that you can begin writing tests, let me know if I can help with that.

MarcelMue commented 6 years ago

I will hopefully get to this next week, I'll try to set up as you described above then. I'd suggest you don't invest time on this right now as it's not urgent yet.