Closed
Bug 1432441
Opened 8 years ago
Closed 8 years ago
XCUITests: Change suggested site label
Categories
(Firefox for iOS :: Build & Test, enhancement)
Tracking
()
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| fxios | 10.5 | --- |
People
(Reporter: isabel_rios, Assigned: isabel_rios)
Details
Attachments
(1 file)
XCUI Search Tests are failing because the suggested label used does not appear now.
- testDoNotShowSuggestionsWhenEnteringURL() Expect predicate exists == true for "foobar2000" Button
- testPromptPresence() Expect predicate exists == true for "foobar2000" Button
Lets change it and use a different one.
This is one of the test affected:
https://github.com/mozilla-mobile/firefox-ios/blob/master/XCUITests/SearchTest.swift#L23
Before when entering foobar we expected "foobar2000" to appear.
Now we can use "foobar meaning" that seems more difficult to change.
These tests are failing in both master and v10.x. So once it is fixed in master we would need to uplift to v10.x branch.
| Assignee | ||
Comment 1•8 years ago
|
||
| Assignee | ||
Comment 2•8 years ago
|
||
Comment on attachment 8944687 [details] [review]
Pull Request
Tests are green (the only red is due to the pdf failure which already has a bug open for fixing it)
https://dashboard.buddybuild.com/apps/57bf25c0f096bc01001e21e0/build/5a6720ccdd8f470001cbe67c#tests
Attachment #8944687 -
Flags: review?(npark)
| Assignee | ||
Updated•8 years ago
|
Whiteboard: [needsuplift]
Updated•8 years ago
|
Attachment #8944687 -
Flags: review?(npark) → review+
| Assignee | ||
Comment 3•8 years ago
|
||
Merged on master: https://github.com/mozilla-mobile/firefox-ios/commit/cb1743ffe7a4457fcad9761d1285e74823eaeb70
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
| Assignee | ||
Comment 4•8 years ago
|
||
Uplifted to v10.x: https://github.com/mozilla-mobile/firefox-ios/commit/59062282c6d5576f3bebb78bb43bf612b28c03fd
Whiteboard: [needsuplift]
You need to log in
before you can comment on or make changes to this bug.
Description
•