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

2 files changed, 10 insertions, 4 deletions

diff --git a/src/angular-mocks.js b/src/angular-mocks.js
index 5b3e6fc3..0d284884 100644
--- a/src/angular-mocks.js
+++ b/src/angular-mocks.js
@@ -748,11 +748,17 @@ function createHttpBackendMock($delegate, $defer) {
 
 
   function createShortMethods(prefix) {
-    angular.forEach(['GET', 'PUT', 'POST', 'DELETE', 'PATCH', 'JSONP'], function(method) {
-     $httpBackend[prefix + method] = function(url, data, headers) {
-       return $httpBackend[prefix](method, url, data, headers)
+    angular.forEach(['GET', 'DELETE', 'JSONP'], function(method) {
+     $httpBackend[prefix + method] = function(url, headers) {
+       return $httpBackend[prefix](method, url, undefined, headers)
      }
     });
+
+    angular.forEach(['PUT', 'POST'], function(method) {
+      $httpBackend[prefix + method] = function(url, data, headers) {
+        return $httpBackend[prefix](method, url, data, headers)
+      }
+    });
   }
 };
 
diff --git a/src/service/http.js b/src/service/http.js
index fec4fc8d..e9c49ec2 100644
--- a/src/service/http.js
+++ b/src/service/http.js
@@ -287,7 +287,7 @@ function $HttpProvider() {
      * @param {Object=} config Optional configuration object
      * @returns {XhrFuture} Future object
      */
-    createShortMethods('get', 'delete', 'head', 'patch', 'jsonp');
+    createShortMethods('get', 'delete', 'head', 'jsonp');
 
     /**
      * @ngdoc method