Skip to content

Fix flaky test reate_visual_review_service_spec.rb:16

What does this MR do and why?

!110687 (diffs) has flaky tests.

Was reported by Gitlab Release Tools bot: https://gitlab.com/gitlab-org/release/tasks/-/issues/5003

The project model was defined as unchanged by let_it_be and later added a last test which changes the project

As such it creates a flaky test flow.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

How to set up and validate locally

Could be reproduced with seed 20774:

rspec ./ee/spec/services/notes/create_visual_review_service_spec.rb --seed 20774

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Dmytro Biryukov

Merge request reports

Loading