test: fix flaky additionalFinalizer in TriggerReconcilerOnAllEventIT by csviri · Pull Request #3434 · operator-framework/java-operator-sdk
AI review requested due to automatic review settings
The phase-1 await could pass prematurely: right after create() the resource already has exactly [ADDITIONAL_FINALIZER], before the operator adds its own FINALIZER. This snapshotted a stale event count, so the phase-2 isEqualTo(eventCount + 1) assertion against a monotonic counter would overshoot and time out. Wait for the operator to add its own finalizer before deleting, so phase 1 reflects the completed add/mark-for-deletion/remove cycle and the baseline is deterministic. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters