Merge pull request #26 from chrisiconolly/master

Updates method interceptor to run last on http requests
nabil-boag · Dec 11, 2015 · 12eaf77 · 12eaf77
2 parents 08bd641 + aaa908b
commit 12eaf77
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 2 deletions.
diff --git a/app/src/js/httpMethodInterceptor.js b/app/src/js/httpMethodInterceptor.js
@@ -117,5 +117,5 @@ angular
   })
 
   .config(['$httpProvider', function ($httpProvider) {
-    $httpProvider.interceptors.push('httpMethodInterceptor');
+    $httpProvider.interceptors.unshift('httpMethodInterceptor');
   }]);
diff --git a/app/src/js/httpMethodInterceptor.spec.js b/app/src/js/httpMethodInterceptor.spec.js
@@ -1,4 +1,4 @@
-/* global describe, beforeEach, module, it, inject, jasmine, expect */
+/* global describe, beforeEach, module, it, inject, jasmine, expect, spyOn */
 
 describe('httpMethodInterceptor', function () {
   var httpMethodInterceptor, httpMethodInterceptorProvider, mockQ,
@@ -206,4 +206,23 @@ describe('httpMethodInterceptor', function () {
         expect(res).toBe(response);
       });
   });
+
+  describe('config sets $httpProvider interceptor', function () {
+    var $httpProvider;
+
+    beforeEach(function () {
+      module(function (_$httpProvider_) {
+        $httpProvider = _$httpProvider_;
+        spyOn($httpProvider.interceptors, 'unshift');
+      });
+      module('ng.httpLoader.httpMethodInterceptor');
+      inject();
+    });
+
+    it('should add to $httpProvider interceptors', function () {
+      expect($httpProvider.interceptors.unshift)
+        .toHaveBeenCalledWith('httpMethodInterceptor');
+    });
+  });
 });
+