Closed
Bug 1514171
Opened 7 years ago
Closed 7 years ago
Follow up Bug 1509383 - Fix the XCUITests affected by that bug
Categories
(Firefox for iOS :: Build & Test, enhancement)
Tracking
()
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| fxios | + | --- |
People
(Reporter: isabel_rios, Assigned: garvan)
Details
With that new implementation: Bug 1509383 - WKURLSchemeHander for all except reader mode
Commit: https://github.com/mozilla-mobile/firefox-ios/commit/8b1450fbeb87f1f559a2f8e42971c715dc96bcaf
These are the following issues with the tests:
- A new tab was named as "home" but now it is "home?#panel=0" is this a desired behaviour? If so, the fix is easy, the tests will be modified accordingly.
Example of failing test:
https://dashboard.buddybuild.com/apps/57bf25c0f096bc01001e21e0/build/5c130ea4b1d5d00001c39ae3/test/00ec1447-b2cb-4747-bfe5-88f7212cabb2/fdbfaf5e-4ee5-49a5-8a47-6eed729d7acd
This is affecting several tests.
- Find in page, large doc tests are failing because now the website is not correctly loaded, please see log here with screenshot9.
https://dashboard.buddybuild.com/apps/57bf25c0f096bc01001e21e0/build/5c130ea4b1d5d00001c39ae3/test/00ec1447-b2cb-4747-bfe5-88f7212cabb2/c6022929-5aae-41fb-b8c6-359740f5e009
Since that website is built here:
https://github.com/mozilla-mobile/firefox-ios/blob/master/XCUITests/WebPagesForTesting.swift
And called as: navigator.openURL("http://localhost:6571/find-in-page-test.html")
We will need to to some changes there...Just guessing I tried calling it as navigator.openURL("internal://local/find-in-page-test.html") but didn't work...
I would need your help to understand how to fix this issue. Thanks!
| Reporter | ||
Comment 1•7 years ago
|
||
Garvan, I'm sure you knew I was going to ping you after this patch ;)
Flags: needinfo?(gkeeley)
Yay for tests.
I am fixing these now.
Assignee: nobody → gkeeley
Flags: needinfo?(gkeeley)
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•