Skip to content

Conversation

@Vinnl
Copy link
Collaborator

@Vinnl Vinnl commented Sep 10, 2025

I also changed the Playwright config to preserve videos/traces of the first run, because I was seeing failures here and wanted to debug. (It used to save the retry, but retries after deleting an account always fail.)

Unfortunately, after adding that change the test kept succeeding. Probably still a good idea to keep that option. (And I'll do another retry.)

@Vinnl Vinnl requested review from codemist and flozia September 10, 2025 20:20
@Vinnl Vinnl self-assigned this Sep 10, 2025
@Vinnl Vinnl added the Review: XS Code review time: up to 30min label Sep 10, 2025
@Vinnl Vinnl force-pushed the MNTOR-4583-moscary-functional-tests branch from ae3a09f to 1ccdb82 Compare September 11, 2025 12:08
Base automatically changed from MNTOR-4583-moscary-functional-tests to main September 11, 2025 12:16
Some tests can't be retried (in this case, deleting the account at
the end of a test run), so having only a trace/video of the retry
is not going to be useful to debug the actual cause of the failure.
@Vinnl Vinnl merged commit d6ffa9a into main Sep 15, 2025
18 checks passed
@Vinnl Vinnl deleted the delete-fxa branch September 15, 2025 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Review: XS Code review time: up to 30min

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants