Skip to content

Remove usage of let_it_be in feature spec

Patrick Bajao requested to merge pb-attempt-fix-flaky-feature-spec into master

What does this MR do and why?

Related to !157006 (comment 1982562623) and #448884 (closed).

Based on the failures on this specific feature spec, code that is not meant to be run inside a transaction are getting run within a transaction even if it's not being called inside a transaction out of the spec.

Theory is that since using let_it_be will wrap the example group within a transaction, that causes the flakiness.

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.

Merge request reports