Skip to content

Commit

Permalink
Test Xcode 10 (#21324)
Browse files Browse the repository at this point in the history
Summary:
This also fixes an issue with tagged commit Circle CI jobs, and avoids running the `publish_npm_package` on every commit. The new config ignores commits on *all branches*, but should still catch git tags.
Pull Request resolved: #21324

Differential Revision: D10041629

Pulled By: hramos

fbshipit-source-id: 9b3295b5fcd614c67a8838ffd49c6a5d6ae7fd86
  • Loading branch information
hramos authored and facebook-github-bot committed Sep 26, 2018
1 parent 2da60a8 commit 544adca
Showing 1 changed file with 36 additions and 4 deletions.
40 changes: 36 additions & 4 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -434,6 +434,32 @@ jobs:
- store_test_results:
path: ~/react-native/reports/junit

# Xcode 10: Runs unit tests on iOS and Apple TV devices
test_objc_xcode10:
<<: *defaults
macos:
xcode: "10.0.0"
steps:
- checkout
- run: *setup-artifacts
- run: *yarn

- run: *boot-simulator-iphone
- run: *boot-simulator-appletv
- run: *brew-install-watchman

- run: *run-objc-ios-tests
- run: *run-objc-tvos-tests

# TODO: Fix these failing tests.
- run: *display-broken-tests-warning
- run: *run-podspec-tests
- run: *run-objc-ios-e2e-tests
- run: *run-objc-tvos-e2e-tests

- store_test_results:
path: ~/react-native/reports/junit

# Runs end to end tests (Detox)
test_detox_end_to_end:
<<: *macos_defaults
Expand Down Expand Up @@ -653,10 +679,6 @@ workflows:
requires:
- checkout_code

# Test JavaScript and Node 10 compatibility
- test_node10:
filters: *filter-ignore-gh-pages

# Test Android
- test_android:
filters: *filter-ignore-gh-pages
Expand All @@ -674,9 +696,19 @@ workflows:
requires:
- checkout_code

# Tooling Compatibility Checks
- test_objc_xcode10:
filters: *filter-ignore-gh-pages
- test_node10:
filters: *filter-ignore-gh-pages

# Only runs on vX.X.X tags if all tests are green
- publish_npm_package:
filters:
# ignore any commit on any branch by default
branches:
ignore: /.*/
# only act on version tags
tags:
only: /v[0-9]+(\.[0-9]+)*(\-rc(\.[0-9]+)?)?/
requires:
Expand Down

0 comments on commit 544adca

Please sign in to comment.