Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Commit 9b4efa7

Browse files
vojtajinaIgorMinar
authored andcommitted
feat(mock.$httpBackend): say which request was expected when unexpected request error
1 parent 4aaa2f7 commit 9b4efa7

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

src/angular-mocks.js

+4-2
Original file line numberDiff line numberDiff line change
@@ -631,8 +631,10 @@ angular.module.ngMock.$HttpBackendProvider = function() {
631631
return method == 'JSONP' ? undefined : xhr;
632632
}
633633
}
634-
throw wasExpected ? Error('No response defined !') :
635-
Error('Unexpected request: ' + method + ' ' + url);
634+
throw wasExpected ?
635+
Error('No response defined !') :
636+
Error('Unexpected request: ' + method + ' ' + url + '\n' +
637+
(expectation ? 'Expected ' + expectation : 'No more request expected'));
636638
}
637639

638640
$httpBackend.when = function(method, url, data, headers) {

test/angular-mocksSpec.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -399,7 +399,7 @@ describe('mocks', function() {
399399
hb.when('GET', '/url1').respond(200, 'content');
400400
expect(function() {
401401
hb('GET', '/xxx');
402-
}).toThrow('Unexpected request: GET /xxx');
402+
}).toThrow('Unexpected request: GET /xxx\nNo more request expected');
403403
});
404404

405405

@@ -506,7 +506,7 @@ describe('mocks', function() {
506506

507507
expect(function() {
508508
hb('GET', '/url2', null, noop, {});
509-
}).toThrow('Unexpected request: GET /url2');
509+
}).toThrow('Unexpected request: GET /url2\nExpected GET /url1');
510510
});
511511

512512

0 commit comments

Comments
 (0)