From 46691c2721735c27426b721d780e8816d502b9f2 Mon Sep 17 00:00:00 2001 From: Igor Minar Date: Tue, 10 Jan 2012 10:17:05 -0800 Subject: fix($http): remove support for PATCH + better whenXXX, expectXXX api - there are too many unknowns about PATCH, so I'm dropping its support until we know that this is actually useful - expectGET, expectHEAD and expectJSON (and the same for whenXXX) should not require response data to be specified --- test/angular-mocksSpec.js | 4 ++-- test/service/httpSpec.js | 12 ------------ 2 files changed, 2 insertions(+), 14 deletions(-) (limited to 'test') diff --git a/test/angular-mocksSpec.js b/test/angular-mocksSpec.js index 153c2e33..073d6cd1 100644 --- a/test/angular-mocksSpec.js +++ b/test/angular-mocksSpec.js @@ -819,9 +819,9 @@ describe('ngMock', function() { }); - describe('expect/when shorcuts', function() { + describe('expect/when shortcuts', function() { angular.forEach(['expect', 'when'], function(prefix) { - angular.forEach(['GET', 'POST', 'PUT', 'DELETE', 'PATCH', 'JSONP'], function(method) { + angular.forEach(['GET', 'POST', 'PUT', 'DELETE', 'JSONP'], function(method) { var shortcut = prefix + method; it('should provide ' + shortcut + ' shortcut method', function() { hb[shortcut]('/foo').respond('bar'); diff --git a/test/service/httpSpec.js b/test/service/httpSpec.js index c6adee9e..d827e941 100644 --- a/test/service/httpSpec.js +++ b/test/service/httpSpec.js @@ -479,18 +479,6 @@ describe('$http', function() { }); - it('should have patch()', function() { - $httpBackend.expect('PATCH', '/url').respond(''); - $http.patch('/url'); - }); - - - it('patch() should allow config param', function() { - $httpBackend.expect('PATCH', '/url', undefined, checkHeader('Custom', 'Header')).respond(''); - $http.patch('/url', {headers: {'Custom': 'Header'}}); - }); - - it('should have post()', function() { $httpBackend.expect('POST', '/url', 'some-data').respond(''); $http.post('/url', 'some-data'); -- cgit v1.2.3