CB-11964 Call clean after plugin install and mock it in tests

bin/templates/cordova/Api.js

@@ -18,6 +18,7 @@
 */
 
 var path = require('path');
+var Q = require('q');
 
 var AndroidProject = require('./lib/AndroidProject');
 var AndroidStudio = require('./lib/AndroidStudio');
@@ -200,6 +201,7 @@ Api.prototype.prepare = function (cordovaProject, prepareOptions) {
  */
 Api.prototype.addPlugin = function (plugin, installOptions) {
     var project = AndroidProject.getProjectFile(this.root);
+    var self = this;
 
     installOptions = installOptions || {};
     installOptions.variables = installOptions.variables || {};
@@ -207,18 +209,22 @@ Api.prototype.addPlugin = function (plugin, installOptions) {
     if (!installOptions.variables.PACKAGE_NAME) {
         installOptions.variables.PACKAGE_NAME = project.getPackageName();
     }
-    
+
     if(this.android_studio === true) {
       installOptions.android_studio = true;
     }
 
-    //CB-11964: Do a clean when installing the plugin code to get around
+    return Q()
-    //the Gradle bug introduced by the Android Gradle Plugin Version 2.2
+        .then(function () {
-    //TODO: Delete when the next version of Android Gradle plugin comes out
+            //CB-11964: Do a clean when installing the plugin code to get around
-    
+            //the Gradle bug introduced by the Android Gradle Plugin Version 2.2
-    this.clean();
+            //TODO: Delete when the next version of Android Gradle plugin comes out
-    return PluginManager.get(this.platform, this.locations, project)
+            return self.clean();
-        .addPlugin(plugin, installOptions)
+        })
+        .then(function () {
+            return PluginManager.get(self.platform, self.locations, project)
+                .addPlugin(plugin, installOptions);
+        })
         .then(function () {
             if (plugin.getFrameworks(this.platform).length === 0) return;
 

spec/unit/Api.spec.js

@@ -23,6 +23,7 @@ var Q = require('q');
 var os = require('os');
 var path = require('path');
 var common = require('cordova-common');
+var rewire = require('rewire');
 
 var AndroidProject = require('../../bin/templates/cordova/lib/AndroidProject');
 var builders = require('../../bin/templates/cordova/lib/builders/builders');
@@ -33,7 +34,8 @@ var FIXTURES = path.join(__dirname, '../e2e/fixtures');
 var FAKE_PROJECT_DIR = path.join(os.tmpdir(), 'plugin-test-project');
 
 describe('addPlugin method', function () {
-    var api, fail, gradleBuilder;
+    var api, fail, gradleBuilder, oldClean;
+    var Api = rewire('../../bin/templates/cordova/Api');
 
     beforeEach(function() {
         var pluginManager = jasmine.createSpyObj('pluginManager', ['addPlugin']);
@@ -43,7 +45,8 @@ describe('addPlugin method', function () {
         var projectSpy = jasmine.createSpyObj('AndroidProject', ['getPackageName', 'write']);
         spyOn(AndroidProject, 'getProjectFile').andReturn(projectSpy);
 
-        var Api = require('../../bin/templates/cordova/Api');
+        oldClean = Api.__get__('Api.prototype.clean');
+        Api.__set__('Api.prototype.clean', Q);
         api = new Api('android', FAKE_PROJECT_DIR);
 
         fail = jasmine.createSpy('fail');
@@ -51,6 +54,10 @@ describe('addPlugin method', function () {
         spyOn(builders, 'getBuilder').andReturn(gradleBuilder);
     });
 
+    afterEach(function () {
+        Api.__set__('Api.prototype.clean', oldClean);
+    });
+
     it('should call gradleBuilder.prepBuildFiles for every plugin with frameworks', function(done) {
         api.addPlugin(new PluginInfo(path.join(FIXTURES, 'cordova-plugin-fake')))
         .catch(fail)