fix: remove MultiOwnerDependentTriggeringIT by csviri · Pull Request #3318 · operator-framework/java-operator-sdk
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Removes the unsupported “multi-owner dependent resource” integration test to avoid confusion, along with its supporting test-only custom resource, reconciler, dependent resource, and spec model classes.
Changes:
- Deleted
MultiOwnerDependentTriggeringITintegration test for multi-owner dependent triggering behavior. - Removed the associated test reconciler, dependent ConfigMap implementation, and CR model classes under the same package.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
operator-framework/src/test/java/io/javaoperatorsdk/operator/dependent/multipleupdateondependent/MultipleOwnerDependentSpec.java |
Removes spec POJO used only by the deleted IT. |
operator-framework/src/test/java/io/javaoperatorsdk/operator/dependent/multipleupdateondependent/MultipleOwnerDependentReconciler.java |
Removes test reconciler used only by the deleted IT. |
operator-framework/src/test/java/io/javaoperatorsdk/operator/dependent/multipleupdateondependent/MultipleOwnerDependentCustomResource.java |
Removes test CustomResource type used only by the deleted IT. |
operator-framework/src/test/java/io/javaoperatorsdk/operator/dependent/multipleupdateondependent/MultipleOwnerDependentConfigMap.java |
Removes dependent resource implementation used only by the deleted IT. |
operator-framework/src/test/java/io/javaoperatorsdk/operator/dependent/multipleupdateondependent/MultiOwnerDependentTriggeringIT.java |
Removes the integration test that was passing but considered unsupported/problematic. |