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
Refactor tests NATS controller reconciler #13324
Refactor tests NATS controller reconciler #13324
Conversation
✔️ 🥰 Documentation preview ready! 🥰 🔨 Explore the source changes: faa5ddd 🔍 Inspect the deploy log: https://app.netlify.com/sites/kyma-project-docs-preview/deploys/6203e33b123b550007994fd6 😎 Browse the preview: https://deploy-preview-13324--kyma-project-docs-preview.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your work on improving the tests 💪 😃
I left quite a lot of comments, but they are not really related to your changes. That means you don't have to work on (all/any) of them. However I wanted to start a discussion on the tests itself, now that I am reading through most of them.
components/eventing-controller/controllers/subscription/nats/reconciler_test.go
Show resolved
Hide resolved
components/eventing-controller/controllers/subscription/nats/reconciler_test.go
Show resolved
Hide resolved
components/eventing-controller/controllers/subscription/nats/reconciler_test.go
Show resolved
Hide resolved
components/eventing-controller/controllers/subscription/nats/reconciler_test.go
Outdated
Show resolved
Hide resolved
components/eventing-controller/controllers/subscription/nats/reconciler_test.go
Show resolved
Hide resolved
components/eventing-controller/controllers/subscription/nats/reconciler_test.go
Outdated
Show resolved
Hide resolved
components/eventing-controller/controllers/subscription/nats/reconciler_test.go
Outdated
Show resolved
Hide resolved
components/eventing-controller/controllers/subscription/nats/reconciler_test.go
Outdated
Show resolved
Hide resolved
components/eventing-controller/controllers/subscription/nats/reconciler_test.go
Show resolved
Hide resolved
components/eventing-controller/controllers/subscription/nats/reconciler_test.go
Outdated
Show resolved
Hide resolved
/lgtm |
/retest |
This is a minor refactoring to to the integration tests in controller/ subscription/nats.