Closed tomasaschan closed 8 months ago
Hi @tomasaschan. Thanks for your PR.
I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test
on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test
label.
I understand the commands that are listed here.
Ping @justinsb :)
/ok-to-test
/lgtm It looks good, and pased the test.
Ping! Could someone please /approve this? Would be nice to get it into master
so we can use this in our operator test suites.
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: atoato88, tomasaschan
The full list of commands accepted by this bot can be found here.
The pull request process is described here
What this PR does / why we need it:
This makes the validator use an
interface
compatible with*testing.T
instead of using*testing.T
directly, in order to allow third-party testing libraries to provide an equivalent implementation instead of the built-in one.See e.g. https://onsi.github.io/ginkgo/#using-third-party-libraries for a use case of this setup.