Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(iOS): open the Simulator app by default. #3304

Merged
merged 7 commits into from
Apr 13, 2022

Conversation

asafkorem
Copy link
Contributor

@asafkorem asafkorem commented Apr 6, 2022

As part of Detox v20 changes.

@d4vidi
Copy link
Collaborator

d4vidi commented Apr 7, 2022

@asafkorem please rebase over wix:next so we could see the actual changes rather than many unrelated ones over 49 files 😆

@asafkorem
Copy link
Contributor Author

asafkorem commented Apr 7, 2022

@asafkorem please rebase over wix:next so we could see the actual changes rather than many unrelated ones over 49 files 😆

The files and commits you see here are next's changes, it was already above this branch, I guess that next was rebased or something so GitHub got a little confused.. Rebasing again anyway 😄

@asafkorem asafkorem force-pushed the feature/open-ios-sim-on-boot branch from 007a309 to 7ecc0f5 Compare April 7, 2022 08:22
@d4vidi
Copy link
Collaborator

d4vidi commented Apr 7, 2022

@asafkorem please rebase over wix:next so we could see the actual changes rather than many unrelated ones over 49 files 😆

The files and commits you see here are next's changes, it was already above this branch, I guess that next was rebased or something so GitHub got a little confused.. Rebasing again anyway 😄

That's legit 😄

@noomorph noomorph force-pushed the feature/open-ios-sim-on-boot branch from 7ecc0f5 to c314757 Compare April 7, 2022 09:50
Copy link
Collaborator

@noomorph noomorph left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd suggest setting zero retries.

@noomorph
Copy link
Collaborator

noomorph commented Apr 7, 2022

@asafkorem, overall, I disapprove the current implementation because it does not react to headless: true | false device option.

@asafkorem
Copy link
Contributor Author

Done @noomorph

@asafkorem asafkorem force-pushed the feature/open-ios-sim-on-boot branch from 4a6ef55 to b40eef7 Compare April 10, 2022 12:29
@asafkorem asafkorem changed the title feat(iOS): open the Simulator application after booting a simulator. feat(iOS): open the Simulator app by default. Apr 10, 2022
@asafkorem asafkorem force-pushed the feature/open-ios-sim-on-boot branch from ec81636 to f64f21c Compare April 11, 2022 11:22
@asafkorem
Copy link
Contributor Author

Pushed another fix (add warning & use --headless on CI) @noomorph.

@noomorph
Copy link
Collaborator

I'll look tomorrow.

detox/index.d.ts Outdated Show resolved Hide resolved
detox/index.d.ts Outdated Show resolved Hide resolved
@noomorph noomorph force-pushed the feature/open-ios-sim-on-boot branch from 43388a5 to f0a1304 Compare April 13, 2022 10:47
@noomorph noomorph merged commit 1a674dd into wix:next Apr 13, 2022
@asafkorem asafkorem deleted the feature/open-ios-sim-on-boot branch April 13, 2022 11:18
@d4vidi d4vidi mentioned this pull request Apr 25, 2022
27 tasks
noomorph pushed a commit that referenced this pull request Apr 27, 2022
BREAKING: opens Simulator app by default unless the headless mode is enabled
noomorph pushed a commit that referenced this pull request May 3, 2022
BREAKING: opens Simulator app by default unless the headless mode is enabled
noomorph pushed a commit that referenced this pull request May 13, 2022
BREAKING: opens Simulator app by default unless the headless mode is enabled
@asafkorem asafkorem restored the feature/open-ios-sim-on-boot branch July 12, 2022 16:31
asafkorem added a commit that referenced this pull request Jul 12, 2022
noomorph added a commit that referenced this pull request Nov 10, 2022
* feat: set minimal Node requirement to 14.5.x
* feat: drop old Jest integrations

BREAKING CHANGE: please upgrade to jest-circus
https://wix.github.io/Detox/docs/guide/jest/

* feat(config): drop default "e2e" value for specs (#3289)

BREAKING CHANGE: Detox won't be appending "e2e" glob to the test runner
if it is not explicitly configured to do so via "specs" property in the
config.

* feat(config): drop skipLegacyWorkersInjection (#3286)

BREAKING CHANGE: Detox will no longer be injecting "-w 1" to Jest as CLI
arguments

* feat(android): boot read-only emulators by default

* feat(iOS): support --headless config (#3304)

BREAKING: opens Simulator app by default unless the headless mode is enabled

* feat: drop Mocha support (#3310)

BREAKING: Detox users should migrate to Jest

* feat(android): remove deprecated native IdlePolicyConfig (#3332)

BREAKING CHANGE

* feat: drop device.appLaunchArgs [options] argument (#3360)

BREAKING: use device.appLaunchArgs.shared for permanent operations

* feat: remove ios.none device type (#3361)

* feat: undeprecate device.relaunchApp() (#3362)

* feat(config): remove legacy dash-based props (#3371)

BREAKING: removes test-runner and runner-config props
from Detox global config. Use testRunner and runnerConfig
instead.

* feat(config): drop support for the all-in-one configuration format (#3386)

BREAKING: please migrate to the new { apps, devices, configurations }
schema that Detox has been already using for more than a year.

* feat: drop jest<27.2.5 support (#3392)

BREAKING: please migrate your projects to Jest 28.x or 27.x

* fix(ios): headless mode

* feat: new global lifecycle for Detox (#3333)

* fix(cli): forward env overrides anyway

* feat(config): add testRunner.forwardEnv

Also, fixes --inspect-brk debugging experience.
Now the entire debug happens in a single test runner process.

* feat(api): detoxInternals.tracing.createEventStream

* feat(config): add reversePorts to app config (#3610)

* fix(Android): pass `headless` config param to the AVD validator. (#3614)

* feat(cli): add reset-lock-file command

* website: add Showcase page

* website: homepage changes

* feat(cli): drop --device-launch-args arg (#3665)

BREAKING: use --device-boot-args instead

Co-authored-by: Asaf Korem <asaf.korem@gmail.com>
Co-authored-by: d4vidi <amit.d4vidi@gmail.com>
Co-authored-by: jasonbariwix <jasonba@wix.com>
Co-authored-by: Yevheniia Hlovatska <33955728+evgeniyaglovatskaya@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants