fix($httpBackend): fix for jsonp requests

Author: IgorMinar

src/angular-mocks.js

@@ -147,8 +147,8 @@ angular.module.ngMock.$Browser = function() {
   };
 
   self.$$scripts = [];
-  self.addJs = function(url, domId, done) {
-    var script = {url: url, id: domId, done: done};
+  self.addJs = function(url, done) {
+    var script = {url: url, done: done};
     self.$$scripts.push(script);
     return script;
   };

src/service/httpBackend.js

@@ -38,7 +38,8 @@ function createHttpBackend($browser, XHR, $browserDefer, callbacks, body, locati
         callbacks[callbackId].data = data;
       };
 
-      var script = $browser.addJs(url.replace('JSON_CALLBACK', callbackId), null, function() {
+      var script = $browser.addJs(url.replace('JSON_CALLBACK', 'angular.callbacks.' + callbackId),
+          function() {
         if (callbacks[callbackId].data) {
           completeRequest(callback, 200, callbacks[callbackId].data);
         } else {

test/angular-mocksSpec.js

@@ -6,16 +6,14 @@ describe('mocks', function() {
 
     describe('addJs', function() {
 
-      it('should store url, id, done', inject(function($browser) {
+      it('should store url, done', inject(function($browser) {
         var url  = 'some.js',
-            id   = 'js-id',
             done = noop;
 
-        $browser.addJs(url, id, done);
+        $browser.addJs(url, done);
 
         var script = $browser.$$scripts.shift();
         expect(script.url).toBe(url);
-        expect(script.id).toBe(id);
         expect(script.done).toBe(done);
       }));
 

test/service/httpBackendSpec.js

@@ -19,7 +19,7 @@ describe('$httpBackend', function() {
 
 
   beforeEach(inject(function($injector) {
-    callbacks = {};
+    callbacks = {counter: 0};
     $browser = $injector.get('$browser');
     fakeBody = {removeChild: jasmine.createSpy('body.removeChild')};
     $backend = createHttpBackend($browser, MockXhr, fakeTimeout, callbacks, fakeBody);
@@ -115,6 +115,9 @@ describe('$httpBackend', function() {
 
   describe('JSONP', function() {
 
+    var SCRIPT_URL = /([^\?]*)\?cb=angular\.callbacks\.(.*)/;
+
+
     it('should add script tag for JSONP request', function() {
       callback.andCallFake(function(status, response) {
         expect(status).toBe(200);
@@ -125,10 +128,10 @@ describe('$httpBackend', function() {
       expect($browser.$$scripts.length).toBe(1);
 
       var script = $browser.$$scripts.shift(),
-          url = script.url.split('?cb=');
+          url = script.url.match(SCRIPT_URL);
 
-      expect(url[0]).toBe('http://example.org/path');
-      callbacks[url[1]]('some-data');
+      expect(url[1]).toBe('http://example.org/path');
+      callbacks[url[2]]('some-data');
       script.done();
 
       expect(callback).toHaveBeenCalledOnce();
@@ -140,7 +143,7 @@ describe('$httpBackend', function() {
       expect($browser.$$scripts.length).toBe(1);
 
       var script = $browser.$$scripts.shift(),
-          callbackId = script.url.split('?cb=')[1];
+          callbackId = script.url.match(SCRIPT_URL)[2];
 
       callbacks[callbackId]('some-data');
       script.done();

test/widgetsSpec.js

@@ -244,7 +244,6 @@ describe('widget', function() {
             $rootScope.tpl = template;
             $rootScope.value = value;
           });
-          $browser.defer.flush();
         };
       }