Skip to content
This repository has been archived by the owner on Feb 24, 2025. It is now read-only.

Fix Sync E2E tests failing on missing Settings #3972

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .maestro/release_tests/backgrounding.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,7 @@ tags:
- tapOn: "Browsing menu"
- assertVisible: "Add Bookmark"
- tapOn: "Add Bookmark"
- tapOn:
id: PrivacyIcon
- tapOn: "Privacy Icon"
- assertVisible: "Protections are ON for this site"
- tapOn: "Done"

Expand All @@ -55,6 +54,7 @@ tags:

# Check menus
- tapOn: "Cancel"
- tapOn: "Browsing menu"
- tapOn: "Settings"
- assertVisible: "Default Browser"
- tapOn: "Done"
Expand Down
6 changes: 2 additions & 4 deletions .maestro/release_tests/content-blocking.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,7 @@ tags:
- pressKey: Enter

- assertVisible: "1 major tracker loaded via script src"
- tapOn:
id: PrivacyIcon
- tapOn: "Privacy Icon"
- assertVisible: "Protections are ON for this site"

- assertVisible: "We blocked Google Ads (Google) from loading tracking requests on this page."
Expand All @@ -47,7 +46,6 @@ tags:

- assertVisible: "1 major tracker loaded via script src"

- tapOn:
id: PrivacyIcon
- tapOn: "Privacy Icon"
- assertVisible: "Protections are OFF for this site"

1 change: 1 addition & 0 deletions .maestro/sync_tests/01_create_account.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ name: 01_create_account
file: ../shared/setup.yaml

# Set Internal User
- tapOn: Browsing menu
- tapOn: Settings
- runFlow:
file: ../shared/set_internal_user_from_settings.yaml
Expand Down
1 change: 1 addition & 0 deletions .maestro/sync_tests/02_login_account.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ name: 02_login_account
file: ../shared/setup.yaml

# Copy Recovery Code
- tapOn: Browsing menu
- tapOn: Settings
- runFlow:
file: ../shared/copy_recovery_code_from_settings.yaml
Expand Down
3 changes: 2 additions & 1 deletion .maestro/sync_tests/03_recover_account.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,8 @@ name: 03_recover_account
file: ../shared/setup.yaml

# Set Internal User
- tapOn: Settings
- tapOn: "Browsing menu"
- tapOn: "Settings"
- runFlow:
file: ../shared/set_internal_user_from_settings.yaml

Expand Down
1 change: 1 addition & 0 deletions .maestro/sync_tests/04_sync_data_setup.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ name: 04_sync_data_setup
text: "Cancel"
commands:
- tapOn: Cancel
- tapOn: Browsing menu
- tapOn: Settings
- runFlow:
file: ../shared/add_login_from_settings.yaml
Expand Down
4 changes: 4 additions & 0 deletions .maestro/sync_tests/05_sync_data_check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ name: 05_sync_data_check
file: ../shared/setup.yaml

# Copy Recovery Code
- tapOn: Browsing menu
- tapOn: Settings
- runFlow:
file: ../shared/copy_recovery_code_from_settings.yaml
Expand All @@ -32,17 +33,20 @@ name: 05_sync_data_check
- assertVisible: Sync & Backup

# Verify bookmarks have been merged
- tapOn: Browsing menu
- tapOn: Settings
- runFlow:
file: ../shared/sync_verify_bookmarks.yaml

# Verify favorites are unified
- tapOn: Done
- tapOn: Browsing menu
- tapOn: Settings
- runFlow:
file: ../shared/sync_verify_unified_favorites.yaml

# Verify logins
- tapOn: Browsing menu
- tapOn: Settings
- runFlow:
file: ../shared/sync_verify_logins.yaml
Expand Down
1 change: 1 addition & 0 deletions .maestro/sync_tests/06_delete_account.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ name: 06_delete_account
file: ../shared/setup.yaml

# Set Internal User
- tapOn: Browsing menu
- tapOn: Settings
- runFlow:
file: ../shared/set_internal_user_from_settings.yaml
Expand Down
Loading