feat(browser.defer): flush should throw exception when queue is empty

angular · Jan 9, 2012 · 63cca9a · 63cca9a
1 parent d47ec77
commit 63cca9a
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/src/angular-mocks.js b/src/angular-mocks.js
@@ -125,6 +125,8 @@ angular.module.ngMock.$Browser = function() {
     } else {
       if (self.deferredFns.length) {
         self.defer.now = self.deferredFns[self.deferredFns.length-1].time;
+      } else {
+        throw Error('No deferred tasks to be flushed');
       }
     }
 

diff --git a/test/angular-mocksSpec.js b/test/angular-mocksSpec.js
@@ -291,6 +291,10 @@ describe('mocks', function() {
       expect(browser.defer.now).toEqual(3);
       expect(log).toEqual('A;B;C;');
     });
+
+    it('should throw an exception if there is nothing to be flushed', function() {
+      expect(function() {browser.defer.flush();}).toThrow('No deferred tasks to be flushed');
+    });
   });
 
 

diff --git a/test/service/deferSpec.js b/test/service/deferSpec.js
@@ -17,7 +17,7 @@ describe('$defer', function() {
     $browser.defer.flush();
     expect(counter).toBe(1);
 
-    $browser.defer.flush(); //does nothing
+    expect(function() {$browser.defer.flush();}).toThrow('No deferred tasks to be flushed');
     expect(counter).toBe(1);
 
     expect($exceptionHandler).not.toHaveBeenCalled();