From c6ea1be0536b9d4564f028554fd1762feba58994 Mon Sep 17 00:00:00 2001 From: Vojta Jina Date: Tue, 10 Jan 2012 23:08:30 -0800 Subject: fix(mock.$httpBackend): resetExpectations should not create new array --- src/angular-mocks.js | 4 ++-- test/angular-mocksSpec.js | 15 +++++++++++++++ 2 files changed, 17 insertions(+), 2 deletions(-) diff --git a/src/angular-mocks.js b/src/angular-mocks.js index 62a37763..9cc0db43 100644 --- a/src/angular-mocks.js +++ b/src/angular-mocks.js @@ -723,8 +723,8 @@ angular.mock.$httpBackendDecorator = function($delegate, $defer) { }; $httpBackend.resetExpectations = function() { - expectations = []; - responses = []; + expectations.length = 0; + responses.length = 0; }; return $httpBackend; diff --git a/test/angular-mocksSpec.js b/test/angular-mocksSpec.js index d7f40bb2..248bb108 100644 --- a/test/angular-mocksSpec.js +++ b/test/angular-mocksSpec.js @@ -780,6 +780,21 @@ describe('mocks', function() { expect(callback).toHaveBeenCalledOnce(); expect(cancelledClb).not.toHaveBeenCalled(); }); + + + it('should not remove definitions', function() { + var cancelledClb = jasmine.createSpy('cancelled'); + + hb.when('GET', '/url').respond(200, 'success'); + hb('GET', '/url', null, cancelledClb); + hb.resetExpectations(); + + hb('GET', '/url', null, callback, {}); + hb.flush(); + + expect(callback).toHaveBeenCalledOnce(); + expect(cancelledClb).not.toHaveBeenCalled(); + }); }); -- cgit v1.2.3