What does this MR do?
- Fix transient error with filtered search lazy author/assignee tokens
Are there points in the code the reviewer needs to double check?
These changes were used in ce-to-ee-9-3-rc2
, https://gitlab.slack.com/archives/C0XM5UU6B/p1497572310860144
Reasoning
I think this happens because of the
author
andassignee
tokens where we swap out the username for an avatar and name
Failing screenshot ❌ Passing screenshot (locally) ✅
When the test fails, it tries to click on the input,
.filtered-search
, but the avatars load in and it accidentally clicks a token instead. I marked the spot, the test tried to click1155.5, 421
which missed
Why was this MR needed?
Transient test failures
As seen in,
Does this MR meet the acceptance criteria?
- Tests
-
Added for this feature/bug -
All builds are passing
-
-
Conform by the merge request performance guides -
Conform by the style guides -
Branch has no merge conflicts with master
(if it does - rebase it please) -
Squashed related commits together