fix(http): default method to GET

Fixes angular#5985 Closes angular#6401
pkozlowski-opensource · Feb 21, 2014 · 6af172a · 6af172a
1 parent 0c9abc3
commit 6af172a
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/src/ng/http.js b/src/ng/http.js
@@ -664,6 +664,7 @@ function $HttpProvider() {
      */
     function $http(requestConfig) {
       var config = {
+        method: 'get',
         transformRequest: defaults.transformRequest,
         transformResponse: defaults.transformResponse
       };

diff --git a/test/ng/httpSpec.js b/test/ng/httpSpec.js
@@ -1120,6 +1120,16 @@ describe('$http', function() {
         expect(callback.mostRecentCall.args[0]).toBe('content');
       }));
 
+      iit('should cache request when cache is provided and no method specified', function () {
+        doFirstCacheRequest();
+
+        $http({url: '/url', cache: cache}).success(callback);
+        $rootScope.$digest();
+
+        expect(callback).toHaveBeenCalledOnce();
+        expect(callback.mostRecentCall.args[0]).toBe('content');
+      });
+
 
       it('should not cache when cache is not provided', function() {
         doFirstCacheRequest();