Support additional dependency declaration format

Previously, `isInstalled` was somewhat naively checking for the presence
of a string in the `build.gradle` file to determine whether or not that
dependency was already linked. I.e.:

`    compile project(':${name}')\n`

…where `name` is replaced with the name of the dependency being checked.

This was inflexible as it only supported that particular format of
`compile` definition. Another, valid `compile` definition follows:

`    compile(project(':example') { … }`

However, this failed the check because it didn't exactly match the
format for which the check was searching the `build.gradle` contents. As
a result, running `react-native link` would incorrectly duplicate the
dependency definition and thus cause a crash upon launching the app.

This change adds an `installPattern` to the object returned from
`makeBuildPatch`, which includes the particular dependency name and is
valid for both `compile` definition formats.

local-cli/link/__fixtures__/android/patchedBuild.gradle

@@ -1,5 +1,8 @@
 dependencies {
     compile project(':test')
+    compile(project(':test2')) {
+      exclude(group: 'org.unwanted', module: 'test10')
+    }
     compile fileTree(dir: "libs", include: ["*.jar"])
     compile "com.android.support:appcompat-v7:23.0.1"
     compile "com.facebook.react:react-native:0.18.+"

local-cli/link/__tests__/android/isInstalled.spec.js

@@ -10,11 +10,12 @@ const projectConfig = {
 };
 
 describe('android::isInstalled', () => {
-  it('should return true when project is already in build.gradle', () =>
+  it('should return true when project is already in build.gradle', () => {
     expect(isInstalled(projectConfig, 'test')).toBeTruthy()
-  );
+    expect(isInstalled(projectConfig, 'test2')).toBeTruthy()
+  });
 
   it('should return false when project is not in build.gradle', () =>
-    expect(isInstalled(projectConfig, 'test2')).toBeFalsy()
+    expect(isInstalled(projectConfig, 'test3')).toBeFalsy()
   );
 });

local-cli/link/__tests__/android/makeBuildPatch.spec.js

@@ -15,4 +15,10 @@ describe('makeBuildPatch', () => {
     const {patch} = makeBuildPatch(name);
     expect(patch).toBe(`    compile project(':${name}')\n`);
   });
+
+  it('should make a correct install check pattern', () => {
+    const {installPattern} = makeBuildPatch(name);
+    const match = `/\\s{4}(compile)(\\(|\\s)(project)\\(\\':${name}\\'\\)(\\)|\\s)/`;
+    expect(installPattern.toString()).toBe(match);
+  });
 });

local-cli/link/android/isInstalled.js

@@ -2,7 +2,6 @@ const fs = require('fs');
 const makeBuildPatch = require('./patches/makeBuildPatch');
 
 module.exports = function isInstalled(config, name) {
-  return fs
-    .readFileSync(config.buildGradlePath)
-    .indexOf(makeBuildPatch(name).patch) > -1;
+  const buildGradle = fs.readFileSync(config.buildGradlePath);
+  return makeBuildPatch(name).installPattern.test(buildGradle);
 };

local-cli/link/android/patches/makeBuildPatch.js

@@ -1,6 +1,11 @@
 module.exports = function makeBuildPatch(name) {
+  const installPattern = new RegExp(
+    `\\s{4}(compile)(\\(|\\s)(project)\\(\\\':${name}\\\'\\)(\\)|\\s)`
+  )
+
   return {
+    installPattern,
     pattern: /[^ \t]dependencies {\n/,
-    patch: `    compile project(':${name}')\n`,
+    patch: `    compile project(':${name}')\n`
   };
 };