fix(mocks): rename mock.animate to ngAnimateMock and ensure it contains all test helper code for ngAnimate

matsko · matsko · commit 4224cd5182bc · 2014-02-06T01:22:14.000-05:00
Closes angular#5822 Closes angular#5917
diff --git a/src/ngMock/angular-mocks.js b/src/ngMock/angular-mocks.js
@@ -757,43 +757,29 @@ angular.mock.TzDate = function (offset, timestamp) {
 angular.mock.TzDate.prototype = Date.prototype;
 /* jshint +W101 */
 
-// TODO(matias): remove this IMMEDIATELY once we can properly detect the
-// presence of a registered module
-var animateLoaded;
-try {
-  angular.module('ngAnimate');
-  animateLoaded = true;
-} catch(e) {}
-
-if(animateLoaded) {
-  angular.module('ngAnimate').config(['$provide', function($provide) {
+angular.mock.animate = angular.module('ngAnimateMock', ['ng'])
+
+  .config(['$provide', function($provide) {
     var reflowQueue = [];
+
     $provide.value('$$animateReflow', function(fn) {
       reflowQueue.push(fn);
       return angular.noop;
     });
-    $provide.decorator('$animate', function($delegate) {
-      $delegate.triggerReflow = function() {
-        if(reflowQueue.length === 0) {
-          throw new Error('No animation reflows present');
-        }
-        angular.forEach(reflowQueue, function(fn) {
-          fn();
-        });
-        reflowQueue = [];
-      };
-      return $delegate;
-    });
-  }]);
-}
 
-angular.mock.animate = angular.module('mock.animate', ['ng'])
-
-  .config(['$provide', function($provide) {
     $provide.decorator('$animate', function($delegate) {
       var animate = {
         queue : [],
         enabled : $delegate.enabled,
+        triggerReflow : function() {
+          if(reflowQueue.length === 0) {
+            throw new Error('No animation reflows present');
+          }
+          angular.forEach(reflowQueue, function(fn) {
+            fn();
+          });
+          reflowQueue = [];
+        }
       };
 
       angular.forEach(['enter','leave','move','addClass','removeClass'], function(method) {
diff --git a/test/ng/compileSpec.js b/test/ng/compileSpec.js
@@ -4490,7 +4490,7 @@ describe('$compile', function() {
 
   describe('$animate animation hooks', function() {
 
-    beforeEach(module('mock.animate'));
+    beforeEach(module('ngAnimateMock'));
 
     it('should automatically fire the addClass and removeClass animation hooks',
       inject(function($compile, $animate, $rootScope) {
diff --git a/test/ng/directive/ngClassSpec.js b/test/ng/directive/ngClassSpec.js
@@ -309,7 +309,7 @@ describe('ngClass animations', function() {
   var body, element, $rootElement;
 
   it("should avoid calling addClass accidentally when removeClass is going on", function() {
-    module('mock.animate');
+    module('ngAnimateMock');
     inject(function($compile, $rootScope, $animate, $timeout) {
       var element = angular.element('<div ng-class="val"></div>');
       var body = jqLite(document.body);
@@ -416,7 +416,7 @@ describe('ngClass animations', function() {
   });
 
   it("should not remove classes if they're going to be added back right after", function() {
-    module('mock.animate');
+    module('ngAnimateMock');
 
     inject(function($rootScope, $compile, $animate) {
       var className;
diff --git a/test/ng/directive/ngIfSpec.js b/test/ng/directive/ngIfSpec.js
@@ -210,7 +210,7 @@ describe('ngIf animations', function () {
     return element;
   }
 
-  beforeEach(module('mock.animate'));
+  beforeEach(module('ngAnimateMock'));
 
   beforeEach(module(function() {
     // we need to run animation on attached elements;
diff --git a/test/ng/directive/ngIncludeSpec.js b/test/ng/directive/ngIncludeSpec.js
@@ -353,7 +353,7 @@ describe('ngInclude', function() {
       };
     }
 
-    beforeEach(module(spyOnAnchorScroll(), 'mock.animate'));
+    beforeEach(module(spyOnAnchorScroll(), 'ngAnimateMock'));
     beforeEach(inject(
         putIntoCache('template.html', 'CONTENT'),
         putIntoCache('another.html', 'CONTENT')));
@@ -589,7 +589,7 @@ describe('ngInclude animations', function() {
     dealoc(element);
   });
 
-  beforeEach(module('mock.animate'));
+  beforeEach(module('ngAnimateMock'));
 
   afterEach(function(){
     dealoc(element);
diff --git a/test/ng/directive/ngRepeatSpec.js b/test/ng/directive/ngRepeatSpec.js
@@ -1149,7 +1149,7 @@ describe('ngRepeat animations', function() {
     return element;
   }
 
-  beforeEach(module('mock.animate'));
+  beforeEach(module('ngAnimateMock'));
 
   beforeEach(module(function() {
     // we need to run animation on attached elements;
diff --git a/test/ng/directive/ngShowHideSpec.js b/test/ng/directive/ngShowHideSpec.js
@@ -73,7 +73,7 @@ describe('ngShow / ngHide animations', function() {
     body.removeAttr('ng-animation-running');
   });
 
-  beforeEach(module('mock.animate'));
+  beforeEach(module('ngAnimateMock'));
 
   beforeEach(module(function($animateProvider, $provide) {
     return function(_$rootElement_) {
diff --git a/test/ng/directive/ngSwitchSpec.js b/test/ng/directive/ngSwitchSpec.js
@@ -223,7 +223,7 @@ describe('ngSwitch animations', function() {
     return element;
   }
 
-  beforeEach(module('mock.animate'));
+  beforeEach(module('ngAnimateMock'));
 
   beforeEach(module(function() {
     // we need to run animation on attached elements;
diff --git a/test/ngAnimate/animateSpec.js b/test/ngAnimate/animateSpec.js
@@ -3,6 +3,7 @@
 describe("ngAnimate", function() {
 
   beforeEach(module('ngAnimate'));
+  beforeEach(module('ngAnimateMock'));
 
 
   it("should disable animations on bootstrap for structural animations even after the first digest has passed", function() {
diff --git a/test/ngRoute/directive/ngViewSpec.js b/test/ngRoute/directive/ngViewSpec.js
@@ -684,7 +684,7 @@ describe('ngView animations', function() {
 
   describe('hooks', function() {
     beforeEach(module('ngAnimate'));
-    beforeEach(module('mock.animate'));
+    beforeEach(module('ngAnimateMock'));
 
     it('should fire off the enter animation',
         inject(function($compile, $rootScope, $location, $timeout, $animate) {
@@ -702,7 +702,7 @@ describe('ngView animations', function() {
 
       var item;
       $templateCache.put('/foo.html', [200, '<div>foo</div>', {}]);
-      element = $compile(html('<ng-view></div>'))($rootScope);
+      element = $compile(html('<div ng-view></div>'))($rootScope);
 
       $location.path('/foo');
       $rootScope.$digest();
@@ -863,7 +863,7 @@ describe('ngView animations', function() {
       };
     }
 
-    beforeEach(module(spyOnAnchorScroll(), 'mock.animate'));
+    beforeEach(module(spyOnAnchorScroll(), 'ngAnimateMock'));
     beforeEach(inject(spyOnAnimateEnter()));
 
     it('should call $anchorScroll if autoscroll attribute is present', inject(

Original file line number	Diff line number	Diff line change
`@@ -210,7 +210,7 @@ describe('ngIf animations', function () {`
`210`	`210`	`return element;`
`211`	`211`	`}`
`212`	`212`
`213`		`- beforeEach(module('mock.animate'));`
	`213`	`+ beforeEach(module('ngAnimateMock'));`
`214`	`214`
`215`	`215`	`beforeEach(module(function() {`
`216`	`216`	`// we need to run animation on attached elements;`
Original file line number	Diff line number	Diff line change
`@@ -1149,7 +1149,7 @@ describe('ngRepeat animations', function() {`
`1149`	`1149`	`return element;`
`1150`	`1150`	`}`
`1151`	`1151`
`1152`		`- beforeEach(module('mock.animate'));`
	`1152`	`+ beforeEach(module('ngAnimateMock'));`
`1153`	`1153`
`1154`	`1154`	`beforeEach(module(function() {`
`1155`	`1155`	`// we need to run animation on attached elements;`
Original file line number	Diff line number	Diff line change
`@@ -223,7 +223,7 @@ describe('ngSwitch animations', function() {`
`223`	`223`	`return element;`
`224`	`224`	`}`
`225`	`225`
`226`		`- beforeEach(module('mock.animate'));`
	`226`	`+ beforeEach(module('ngAnimateMock'));`
`227`	`227`
`228`	`228`	`beforeEach(module(function() {`
`229`	`229`	`// we need to run animation on attached elements;`