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

Only reference libXCTestBundleInject.dylib if it exists #460

Conversation

RainNapper
Copy link
Contributor

@RainNapper RainNapper commented Oct 7, 2020

App will crash if this dylib does not exist in the host app bundle. This dylib is not required for tests to run.

ravimandala
ravimandala previously approved these changes Oct 7, 2020
Copy link
Contributor

@ravimandala ravimandala left a comment

Choose a reason for hiding this comment

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

LGTM.

ob
ob previously approved these changes Oct 7, 2020
Copy link
Member

@ob ob left a comment

Choose a reason for hiding this comment

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

Thanks for fixing my buggy code :-)

bp/src/SimulatorHelper.m Outdated Show resolved Hide resolved
Copy link
Contributor

@ravimandala ravimandala left a comment

Choose a reason for hiding this comment

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

LGTM.

@ravimandala ravimandala merged commit db09487 into MobileNativeFoundation:master Oct 9, 2020
ravimandala pushed a commit to ravimandala/bluepill that referenced this pull request Oct 9, 2020
…Foundation#460)

App will crash if this "optional" dylib does not exist in the host app bundle. Fixing it by making it truly optional.
AhmedEid added a commit to tinyspeck/bluepill that referenced this pull request Nov 18, 2020
* master:
  Reference libXCTestBundleInject.dylib only if it exists (MobileNativeFoundation#460)
  Record video using simctl (MobileNativeFoundation#441)
  Xcode 12.0 support (MobileNativeFoundation#458)
  Retry app crash tests and consider then non-fatal if they pass (MobileNativeFoundation#456)
  Do not retry crashed tests (MobileNativeFoundation#443)
  Fixing minor issues and doing some minor refactoring/clean-up (MobileNativeFoundation#444)
  Xcode 11.5 support (MobileNativeFoundation#447)
  Xcode 11.4 support (MobileNativeFoundation#446)
  Add a flag to disable Xcode version check failure (MobileNativeFoundation#436)
  Support sidecar applications (MobileNativeFoundation#438)
  Added a few more tests to mock test failure scenarios and fixed a few final Exit Status issues  (MobileNativeFoundation#430)
  make retry checks more resilient to prevent infinite retries (MobileNativeFoundation#432)
  Xcode 11.3 support (MobileNativeFoundation#426)
  Supporting tests and app hosts in subfolders (MobileNativeFoundation#419)

# Conflicts:
#	bp/src/SimulatorHelper.m
#	bp/tests/SimulatorHelperTests.m
@RainNapper RainNapper deleted the mmark/bluepill/xc12-fix branch July 6, 2021 17:50
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