These tests are disabled and currently not running, the idea is to convert them to XCUITests and have them enabled, up and running
Created attachment 8848487 [details] [review] Link to Github pull-request: https://github.com/mozilla-mobile/firefox-ios/pull/2532
Comment on attachment 8848487 [details] [review] Link to Github pull-request: https://github.com/mozilla-mobile/firefox-ios/pull/2532 There is question about the issue removing the NavigationTests file, which class is called by Global.swift, seems it is possible to replace it by another class existing in the UI since the tests pass, but confirmation would be needed and also to understand how it works. But in the meantime if you could please do a first review to check if the rest looks fine, that would help! Thanks!
Comment on attachment 8848487 [details] [review] Link to Github pull-request: https://github.com/mozilla-mobile/firefox-ios/pull/2532 Once the doubt has been answered and the Global.swift can be modified as in the PR, asking also for review to jhugman. A new method has been added to FxScreenGraph and would be good to have your input and opinion about modifying the tests with the new scenes added to FxScreenGraph by you or leave them as they are. Thanks!
Comment on attachment 8848487 [details] [review] Link to Github pull-request: https://github.com/mozilla-mobile/firefox-ios/pull/2532 Other than one minor comment, squash commit, it looks good to me.
I've added my review in the PR.
Hi James, sorry for asking again but have some questions on the PR that would need your help to answer and so modify the PR with the requested changes. I know you are super busy with the push stuff but if you could please have a look whenever you have some time I would really appreciate it. This and the ReaderViewUI test suite are the only left to convert. Thanks! :)
I've added some further comments on the PR. Looking good!
Thanks for the review! I tried to fix them. Some comments added and a question regarding one accessibility ID I did not found and had to define. Hopefully it would be better now and would not take too much more ouf your time. Thanks again for your help!
Merged https://github.com/mozilla-mobile/firefox-ios/commit/6b6f772e17c63ecca190d7a849ef2fcfc0c6237b Thanks jhugman for your help!