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

Convert tests to run on new framework #178

Closed
10 of 12 tasks
leochr opened this issue May 17, 2021 · 1 comment
Closed
10 of 12 tasks

Convert tests to run on new framework #178

leochr opened this issue May 17, 2021 · 1 comment
Assignees
Labels

Comments

@leochr
Copy link
Member

leochr commented May 17, 2021

Existing tests are not compatible with operator-sdk 1.x. Convert existing tests to one of the new testing framework (kuttl, scorecard, env-test). Tests related to kAppNav integration and servicebinding doesn't have to be ported, but we still want to keep generating Application definition)

e2e tests at https://github.com/application-stacks/runtime-component-operator/tree/master/test/e2e :

  • runtime_basic.go - Jason
  • runtime_pullpolicy.go - Bradley
  • runtime_probe.go - Iain
  • runtime_autoscaling.go - Iain
  • runtime_imagestream.go - Iain
  • runtime_knative.go - Iain
  • runtime_storage.go -Iain
  • runtime_servicemonitor.go - Iain
  • runtime_test.go - I don't believe this is actually a test, but rather the entry point for e2e tests (of all framework?)
  • runtime_kappnav.go - kAppNav integration will be removed in v1, but operator will continue to generate Application definition. Operator will need to be enhanced to add this support. So we don't need to convert this now.
  • runtime_certmanager.go - Cert manager support will change in v1, so we don't need to convert this now.
  • runtime_servicebinding.go - Service binding support will change in v1, so we don't need to convert this now. - handled via Add tests for Service bindings #239

Unit tests: handled via #237

@yongja79
Copy link
Collaborator

The script to run the tests in travis will also need to be updated e2e.sh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants