Swap from timeouts to threading to manage CLI runtime in specs
What does this MR do and why?
Harden Internal Events CLI specs against flakiness (!140156 - merged) was meant to address some recurring flakiness in the internal events CLI specs. It helped, but the root problem still exists. This MR swaps from making assertions after a fixed timeout to waiting for the assertion to become true. The specs are more stable and run faster as a result.
Related issues:
- Failure in spec/scripts/internal_events/cli_spe... (#435795 - closed)
- Failure in spec/scripts/internal_events/cli_spe... (#437007 - closed)
- Failure in spec/scripts/internal_events/cli_spe... (#435606 - closed)
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
How to set up and validate locally
bundle exec rspec spec/scripts/internal_events/cli_spec.rb