aboutsummaryrefslogtreecommitdiffstats
path: root/test/ng/compileSpec.js
diff options
context:
space:
mode:
authorMatias Niemelä2014-01-22 19:21:05 -0500
committerMatias Niemelä2014-02-06 01:21:41 -0500
commit906fdad0f95465842e336e057ea97d0633712189 (patch)
tree0993034fe2ce43ad090203c7d2390662536d1d91 /test/ng/compileSpec.js
parenta8c1d9c97804f7df98150456c2702fd39e3a22f0 (diff)
downloadangular.js-906fdad0f95465842e336e057ea97d0633712189.tar.bz2
fix(mocks): remove usage of $animate.flushNext in favour of queing
The flushNext method of testing is difficult and highly coupled with the behavior of ngAnimate's $animate workflow. It is much better instead to just queue all $animate animation calls into a queue collection which is available on the $animate service when mock.animate is included as a module within test code.
Diffstat (limited to 'test/ng/compileSpec.js')
-rwxr-xr-xtest/ng/compileSpec.js20
1 files changed, 12 insertions, 8 deletions
diff --git a/test/ng/compileSpec.js b/test/ng/compileSpec.js
index 557fb85c..c8cb70f8 100755
--- a/test/ng/compileSpec.js
+++ b/test/ng/compileSpec.js
@@ -4506,8 +4506,9 @@ describe('$compile', function() {
$rootScope.val2 = 'rice';
$rootScope.$digest();
- data = $animate.flushNext('addClass');
- expect(data.params[1]).toBe('ice rice');
+ data = $animate.queue.shift();
+ expect(data.event).toBe('addClass');
+ expect(data.args[1]).toBe('ice rice');
expect(element.hasClass('ice')).toBe(true);
expect(element.hasClass('rice')).toBe(true);
@@ -4516,10 +4517,12 @@ describe('$compile', function() {
$rootScope.val2 = 'dice';
$rootScope.$digest();
- data = $animate.flushNext('removeClass');
- expect(data.params[1]).toBe('rice');
- data = $animate.flushNext('addClass');
- expect(data.params[1]).toBe('dice');
+ data = $animate.queue.shift();
+ expect(data.event).toBe('removeClass');
+ expect(data.args[1]).toBe('rice');
+ data = $animate.queue.shift();
+ expect(data.event).toBe('addClass');
+ expect(data.args[1]).toBe('dice');
expect(element.hasClass('ice')).toBe(true);
expect(element.hasClass('dice')).toBe(true);
@@ -4529,8 +4532,9 @@ describe('$compile', function() {
$rootScope.val2 = '';
$rootScope.$digest();
- data = $animate.flushNext('removeClass');
- expect(data.params[1]).toBe('ice dice');
+ data = $animate.queue.shift();
+ expect(data.event).toBe('removeClass');
+ expect(data.args[1]).toBe('ice dice');
expect(element.hasClass('ice')).toBe(false);
expect(element.hasClass('dice')).toBe(false);