Updated CLI to handle different versions of react and react-test-rend…

…erer
facebook · Mar 23, 2017 · ac37fa4 · ac37fa4
1 parent 0382460
commit ac37fa4
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/local-cli/init/init.js b/local-cli/init/init.js
@@ -60,10 +60,12 @@ function init(projectDir, argsOrName) {
  */
 function generateProject(destinationRoot, newProjectName, options) {
   var reactNativePackageJson = require('../../package.json');
-  var { peerDependencies } = reactNativePackageJson;
+  var { devDependencies, peerDependencies } = reactNativePackageJson;
   if (!peerDependencies) {
     console.error('Missing React peer dependency in React Native\'s package.json. Aborting.');
     return;
+  } else if (!devDependencies) {
+    console.error('Missing react-test-renderer dev dependency in React Native\'s package.json. Aborting.');
   }
 
   var reactVersion = peerDependencies.react;
@@ -72,6 +74,12 @@ function generateProject(destinationRoot, newProjectName, options) {
     return;
   }
 
+  var reactTestRendererVersion = devDependencies['react-test-renderer'];
+  if (!reactVersion) {
+    console.error('Missing react-test-renderer dev dependency in React Native\'s package.json. Aborting.');
+    return;
+  }
+
   const yarnVersion =
     (!options.npm) &&
     yarn.getYarnVersionIfAvailable() &&
@@ -88,7 +96,7 @@ function generateProject(destinationRoot, newProjectName, options) {
   }
   if (!options['skip-jest']) {
     const jestDeps = (
-      `jest babel-jest babel-preset-react-native react-test-renderer@${reactVersion}`
+      `jest babel-jest babel-preset-react-native react-test-renderer@${reactTestRendererVersion}`
     );
     if (yarnVersion) {
       console.log('Adding Jest...');