diff --git a/.circleci/Dockerfiles/scripts/run-ci-e2e-tests.sh b/.circleci/Dockerfiles/scripts/run-ci-e2e-tests.sh index a979d3abc..cede0e977 100755 --- a/.circleci/Dockerfiles/scripts/run-ci-e2e-tests.sh +++ b/.circleci/Dockerfiles/scripts/run-ci-e2e-tests.sh @@ -22,8 +22,6 @@ AVD_UUID=$(< /dev/urandom tr -dc 'a-zA-Z0-9' | fold -w 8 | head -n 1) ANDROID_NPM_DEPS="appium@1.5.1 mocha@2.4.5 wd@0.3.11 colors@1.0.3 pretty-data2@0.40.1" CLI_PACKAGE="$ROOT/react-native-cli/react-native-cli-*.tgz" PACKAGE="$ROOT/react-native-*.tgz" -# Version of react-native-dummy to test against -REACT_DUMMY_PLATFORM=react-native-dummy@0.1.0 # solve issue with max user watches limit echo 65536 | tee -a /proc/sys/fs/inotify/max_user_watches @@ -250,20 +248,6 @@ function e2e_suite() { return 1 fi - if ! retry "$RETRY_COUNT" npm install --save "$REACT_DUMMY_PLATFORM" --silent >> /dev/null - then - echo "Failed to install react-native-dummy" - echo "Most common reason is npm registry connectivity, try again" - return 1 - fi - - if ! react-native bundle --max-workers 1 --platform dummy --dev true --entry-file index.js --bundle-output dummy-bundle.js - then - echo "Could not build dummy bundle" - return 1 - fi - fi - # directory cleanup rm "$IOS_MARKER" rm "$ANDROID_MARKER" diff --git a/jest/__tests__/hasteImpl-test.js b/jest/__tests__/hasteImpl-test.js index 9cfb5a7ae..9cab29a52 100644 --- a/jest/__tests__/hasteImpl-test.js +++ b/jest/__tests__/hasteImpl-test.js @@ -11,23 +11,9 @@ 'use strict'; const path = require('path'); -const fs = require('fs'); const {getHasteName} = require('../hasteImpl'); -// RNPM currently does not support plugins when using Yarn Plug 'n Play -const testIfNotPnP = fs.existsSync( - path.join( - __dirname, - '../..', - 'node_modules', - 'react-native-dummy', - 'package.json', - ), -) - ? test - : test.skip; - function getPath(...parts) { return path.join(__dirname, '..', '..', ...parts); } @@ -60,24 +46,6 @@ it('returns the correct haste name for a file with a platform suffix', () => { } }); -testIfNotPnP( - 'returns the correct haste name for a file with an out-of-tree platform suffix', - () => { - for (const platform of ['dummy']) { - expect( - getHasteName( - getPath( - 'Libraries', - 'Components', - 'AccessibilityInfo', - `AccessibilityInfo.${platform}.js`, - ), - ), - ).toEqual('AccessibilityInfo'); - } - }, -); - it('returns the correct haste name for a file with a flow suffix', () => { expect( getHasteName( diff --git a/package.json b/package.json index b88c3130a..a70cf91ef 100644 --- a/package.json +++ b/package.json @@ -141,7 +141,6 @@ "mkdirp": "^0.5.1", "prettier": "1.16.4", "react": "16.8.3", - "react-native-dummy": "0.2.0", "react-test-renderer": "16.8.3", "shelljs": "^0.7.8", "yargs": "^9.0.0",