Merge pull request #437 from jcesarmobile/CB-14038

CB-14038 (android): fix false positive detecting project type

I agree @jcesarmobile, it can wait for a future release.

Author: infil00p

bin/templates/cordova/lib/AndroidStudio.js

@@ -6,37 +6,6 @@
 
 /* jshint esnext: false */
 
-var path = require('path');
-var fs = require('fs');
-var CordovaError = require('cordova-common').CordovaError;
-
 module.exports.isAndroidStudioProject = function isAndroidStudioProject (root) {
-    var eclipseFiles = ['AndroidManifest.xml', 'libs', 'res'];
-    var androidStudioFiles = ['app', 'app/src/main'];
-
-    // assume it is an AS project and not an Eclipse project
-    var isEclipse = false;
-    var isAS = true;
-
-    if (!fs.existsSync(root)) {
-        throw new CordovaError('AndroidStudio.js:inAndroidStudioProject root does not exist: ' + root);
-    }
-
-    // if any of the following exists, then we are not an ASProj
-    eclipseFiles.forEach(function (file) {
-        if (fs.existsSync(path.join(root, file))) {
-            isEclipse = true;
-        }
-    });
-
-    // if it is NOT an eclipse project, check that all required files exist
-    if (!isEclipse) {
-        androidStudioFiles.forEach(function (file) {
-            if (!fs.existsSync(path.join(root, file))) {
-                console.log('missing file :: ' + file);
-                isAS = false;
-            }
-        });
-    }
-    return (!isEclipse && isAS);
+    return true;
 };

spec/fixtures/android_project/AndroidManifest.xml

@@ -8,9 +8,4 @@ describe('AndroidStudio module', function () {
         var isAndStud = AndroidStudio.isAndroidStudioProject(root);
         expect(isAndStud).toBe(true);
     });
-    it('should return false non Android Studio project', function () {
-        var root = path.join(__dirname, '../fixtures/android_project/');
-        var isAndStud = AndroidStudio.isAndroidStudioProject(root);
-        expect(isAndStud).toBe(false);
-    });
 });