diff options
| author | corrupt | 2013-10-18 15:42:55 +0300 | 
|---|---|---|
| committer | Igor Minar | 2013-11-26 12:36:41 -0800 | 
| commit | 68ceb17272bdd2ebc838565070973d93704f4427 (patch) | |
| tree | 47a875cbc4efc138bf780b133c09436dfcc6bec6 /test/ng/httpBackendSpec.js | |
| parent | 5bd6596856c9f88c0ce3d96ff1ed4daa082cbe29 (diff) | |
| download | angular.js-68ceb17272bdd2ebc838565070973d93704f4427.tar.bz2 | |
chore($httpBackend): preserve original non-zero http status code for file:// apps
Previously if an app was running from file:// origin we would always return either
http 200 or 404 depending on whether the response was present.
This changes the behavior so that we do this only if the protocol of the request
(not the origin) is file:// and only if the status code is 0.
Closes #4436
Closes #4587
Closes #4514
Diffstat (limited to 'test/ng/httpBackendSpec.js')
| -rw-r--r-- | test/ng/httpBackendSpec.js | 56 | 
1 files changed, 52 insertions, 4 deletions
| diff --git a/test/ng/httpBackendSpec.js b/test/ng/httpBackendSpec.js index 0748d116..8c843d2a 100644 --- a/test/ng/httpBackendSpec.js +++ b/test/ng/httpBackendSpec.js @@ -436,13 +436,61 @@ describe('$httpBackend', function() {      it('should convert 0 to 404 if no content - relative url', function() { -      $backend = createHttpBackend($browser, MockXhr, null, null, null, 'file'); +      var originalUrlParsingNode = urlParsingNode; + +      //temporarily overriding the DOM element to pretend that the test runs origin with file:// protocol +      urlParsingNode = { +        hash : "#/C:/", +        host : "", +        hostname : "", +        href : "file:///C:/base#!/C:/foo", +        pathname : "/C:/foo", +        port : "", +        protocol : "file:", +        search : "", +        setAttribute: angular.noop +      }; -      $backend('GET', '/whatever/index.html', null, callback); -      respond(0, ''); +      try { + +        $backend = createHttpBackend($browser, MockXhr); + +        $backend('GET', '/whatever/index.html', null, callback); +        respond(0, ''); + +        expect(callback).toHaveBeenCalled(); +        expect(callback.mostRecentCall.args[0]).toBe(404); + +      } finally { +        urlParsingNode = originalUrlParsingNode; +      } +    }); + + +    it('should return original backend status code if different from 0', function () { +      $backend = createHttpBackend($browser, MockXhr); + +      // request to http:// +      $backend('POST', 'http://rest_api/create_whatever', null, callback); +      respond(201, '');        expect(callback).toHaveBeenCalled(); -      expect(callback.mostRecentCall.args[0]).toBe(404); +      expect(callback.mostRecentCall.args[0]).toBe(201); + + +      // request to file:// +      $backend('POST', 'file://rest_api/create_whatever', null, callback); +      respond(201, ''); + +      expect(callback).toHaveBeenCalled(); +      expect(callback.mostRecentCall.args[0]).toBe(201); + +      // request to file:// with HTTP status >= 300 +      $backend('POST', 'file://rest_api/create_whatever', null, callback); +      respond(503, ''); + +      expect(callback).toHaveBeenCalled(); +      expect(callback.mostRecentCall.args[0]).toBe(503);      });    });  }); | 
