Commit 3f7e4e4
authored
Fix flaky bookmark tests (#7809)
Task/Issue URL:
https://app.asana.com/1/137249556945/project/1202552961248957/task/1213430504228545?focus=true
### Description
Fix flaky Maestro tests for bookmarks by removing step to hide keyboard
### Steps to test this PR
_Feature 1_
- [x] Check [release
tests](https://app.maestro.dev/project/proj_01htg54rdtfwx8rgbzv03cxkpf/maestro-test/app/app_01hkqhj1thevwtn9ym8a2ctn2r/upload/mupload_01kja2htwrfr8s79saxhmeta0g?sort=name)
pass in Maestro. In particular:
- [x] [ReleaseTest: Bookmarks can be added and
deleted](https://app.maestro.dev/project/proj_01htg54rdtfwx8rgbzv03cxkpf/maestro-test/flow/run_01kja2hvageaqtmsb82h954nat)
- [x] [ReleaseTest: Bookmark open and back
navigation](https://app.maestro.dev/project/proj_01htg54rdtfwx8rgbzv03cxkpf/maestro-test/flow/run_01kja2hv6sesh88cr4mxykf19f)
- [x] [ReleaseTest: Bookmark open in folder and back
navigation](https://app.maestro.dev/project/proj_01htg54rdtfwx8rgbzv03cxkpf/maestro-test/flow/run_01kja2hv8mfgh8bvjtcgjmwfsm)
### UI changes
n/a
<!-- CURSOR_SUMMARY -->
---
> [!NOTE]
> **Low Risk**
> Test-only changes that adjust UI automation steps; no production code
paths or data handling are affected.
>
> **Overview**
> Stabilizes the Maestro bookmark release tests by replacing the shared
`click_on_menu_button` subflow with direct `tapOn` interactions on
`browserMenuImageView` when opening the browser menu.
>
> This updates three bookmark flows (add/delete, open+back, and folder
navigation) to remove conditional logic/keyboard handling from the
menu-opening step, reducing flakiness during menu access.
>
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
07fad30. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->1 parent 53676b2 commit 3f7e4e4
File tree
3 files changed
+12
-6
lines changed- .maestro/bookmarks
3 files changed
+12
-6
lines changedLines changed: 4 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
16 | 16 | | |
17 | 17 | | |
18 | 18 | | |
19 | | - | |
| 19 | + | |
| 20 | + | |
20 | 21 | | |
21 | 22 | | |
22 | 23 | | |
23 | 24 | | |
24 | | - | |
| 25 | + | |
| 26 | + | |
25 | 27 | | |
26 | 28 | | |
27 | 29 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
27 | | - | |
| 27 | + | |
| 28 | + | |
28 | 29 | | |
29 | 30 | | |
30 | 31 | | |
31 | 32 | | |
32 | | - | |
| 33 | + | |
| 34 | + | |
33 | 35 | | |
34 | 36 | | |
35 | 37 | | |
| |||
Lines changed: 4 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
21 | | - | |
| 21 | + | |
| 22 | + | |
22 | 23 | | |
23 | 24 | | |
24 | 25 | | |
| |||
29 | 30 | | |
30 | 31 | | |
31 | 32 | | |
32 | | - | |
| 33 | + | |
| 34 | + | |
33 | 35 | | |
34 | 36 | | |
35 | 37 | | |
| |||
0 commit comments