Data Sync User
Data Sync User
➤ Clare So commented: ih-codes The issue could be specific to the server. 😌 Here are the steps to run the sync integration test that crashes Firefox ~60% of the...
➤ ih-codes commented: Update: Clare So has made a PR https://github.com/mozilla-mobile/firefox-ios/pull/21309 ( https://github.com/mozilla-mobile/firefox-ios/pull/21309|smart-link ) which may address the issue, and she will continue to monitor.
➤ Clare So commented: The issue can’t be reproduced under the FullFunctionalTestPlan but is reproducible under SyncIntegrationTestPlan. Let me take a look at the test plan.
➤ Clare So commented: More detailed (manual) reproduction steps: 1. Run one sync integration test on the command line that involves signing in. cd firefox-ios/firefox-ios-tests/Tests/SyncIntegrationTests pipenv run pytest test_integration.py::test_sync_bookmark_from_device 2....
➤ Matt Reagan commented: Follow-up note: fixes were previously implemented for test cases 1-3 (includes Bugzilla) as well as correcting the API misuse of the MessageAlert completion handler (see ticket...
➤ Laurie Marceau commented: Did a quick historical overview of the changes related to JS crashes. We’ve been trying to tackle those JS alert related issues since v110. Here’s the...
➤ Isabella commented: When the fix has been validated in production, there is a follow up task here (https://mozilla-hub.atlassian.net/browse/FXIOS-11031 ( https://mozilla-hub.atlassian.net/browse/FXIOS-11031|smart-link ) ) to investigate the mysterious JS Alert completion...
➤ Andrei Bodea commented: Hello, Isabella Laurie Marceau Matt Reagan Hello, anything for QA in order to verify this issue?
➤ Isabella commented: cc Laurie Marceau Can you let Andrei Bodea know how and what to test? I think you needed to enable a feature flag? 🤔
➤ Laurie Marceau commented: Andrei Bodea There’s 6 use cases listed in the description of this task. None of those should crash. And like commented here ( https://mozilla-hub.atlassian.net/browse/FXIOS-10214?focusedCommentId=996605 ), the...