Skip to content

Conversation

@coreycb
Copy link
Collaborator

@coreycb coreycb commented Nov 24, 2025

I've submitted this on behalf of @VladLucaci

Copy link
Collaborator Author

@coreycb coreycb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@VladLucaci I added a couple of comments. Can you take a look?
It would also be good if we could add explanations to the commit message for existing tests that were updated.

@@ -1,56 +1,50 @@
appId: net.thunderbird.android.debug
appId: net.thunderbird.android.beta
Copy link
Collaborator Author

@coreycb coreycb Nov 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should the appId's remain debug or is there a good reason to switch to beta?

#### Open display settings
#- tapOn:
# id: "android:id/title"
# index: 0
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a lot of yaml commented out throughout these files. Is that expected?

@wmontwe wmontwe added merge block: soft freeze PR to main is blocked: risky code or feature flag enablement must wait until soft freeze lifts. and removed merge block: soft freeze PR to main is blocked: risky code or feature flag enablement must wait until soft freeze lifts. labels Dec 4, 2025
@wmontwe wmontwe changed the title test: Add new Maestro tests test: add new Maestro tests Dec 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants