Closed sbueringer closed 2 years ago
/assign (already have a lot of experience with CAPI unit tests and the controller-runtime test framework)
/unassign
Didn't have time for this. So let's see if somebody wants to pick it up until the next minor release
Related to this slack conversation: https://kubernetes.slack.com/archives/CFKJB65G9/p1623237546041600
@macaptain just fyi, this is the corresponding issue :)
Are you currently on it?
@sbueringer, yes I was hoping to have a go this week.
/assign
@macaptain Perfect, great!
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs.
This bot triages issues and PRs according to the following rules:
lifecycle/stale
is appliedlifecycle/stale
was applied, lifecycle/rotten
is appliedlifecycle/rotten
was applied, the issue is closedYou can:
/remove-lifecycle stale
/lifecycle rotten
/close
Please send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle stale
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.
This bot triages issues and PRs according to the following rules:
lifecycle/stale
is appliedlifecycle/stale
was applied, lifecycle/rotten
is appliedlifecycle/rotten
was applied, the issue is closedYou can:
/remove-lifecycle rotten
/close
Please send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle rotten
@macaptain are you still on this?
@tobiasgiese, I completed an initial implementation of mocking the OpenStack API to support unit tests in CAPO (#935).
Per the issue description:
After the initial implementation I would create a list of additional unit tests for follow-up PRs, which I would then flag with help wanted / good first issue.
I expect we could do this now. @mdbooth had a really promising draft #1030 mocking out the interfaces in the compute module which would also help!
/unassign
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.
This bot triages issues and PRs according to the following rules:
lifecycle/stale
is appliedlifecycle/stale
was applied, lifecycle/rotten
is appliedlifecycle/rotten
was applied, the issue is closedYou can:
/reopen
/remove-lifecycle rotten
Please send feedback to sig-contributor-experience at kubernetes/community.
/close
@k8s-triage-robot: Closing this issue.
/kind feature
We now have e2e / integration / conformance tests but basically zero unit tests. I think we should change that. I think we need at least unit tests on 2 layers:
I would start with an initial implementation to establish patterns how we do unit tests for those layers
Some considerations:
Notes:
After the initial implementation I would create a list of additional unit tests for follow-up PRs, which I would then flag with help wanted / good first issue.