Closed Bug 1162999 Opened 9 years ago Closed 9 years ago

Need to skip first-run in UITests

Categories

(Firefox for iOS :: Build & Test, defect)

ARM
iOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: aaronmt, Assigned: bnicholson)

Details

Attachments

(1 file)

In startup we can tap 'Browse' but it would be nice to skip the first-run all-together.
Couple of ideas for resolving this:

1. I've had problems in the past where tests were not being properly run because of logic that was being executed in the AppDelegate every time a test case started up. A solution would be to create a Test AppDelegate and have a build flag in main.m that would direct the app to the correct app delegate. The nice thing about this solution is that we don't add any test exceptions to the production code.

2. Another solution would be to hide certain functionality in app start up behind build flags that would be configured to avoid certain logic from running on app launch.
Assignee: nobody → bnicholson
Status: NEW → ASSIGNED
Attachment #8606008 - Flags: review?(sleroux)
Comment on attachment 8606008 [details] [review]
Link to Github pull-request: https://github.com/mozilla/firefox-ios/pull/467

Looks good!
Attachment #8606008 - Flags: review?(sleroux) → review+
Not related to this patch but there seems to be 2 failing tests:

https://www.dropbox.com/s/s2rla5x33ek8fc1/Screenshot%202015-05-15%2008.32.15.png?dl=0
(In reply to Stephan Leroux from comment #4)
> Not related to this patch but there seems to be 2 failing tests:
> 
> https://www.dropbox.com/s/s2rla5x33ek8fc1/Screenshot%202015-05-15%2008.32.15.
> png?dl=0

Yeah, wesj mentioned that too -- somehow related to the on-screen keyboard being enabled. I'll look into it.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: