Merge branch 'jhamon-throw-error-when-installed-multiple-times'

lib/mock-ajax.js

@@ -396,6 +396,10 @@ getJasmineRequireObj().MockAjax = function($ajax) {
       mockAjaxFunction = $ajax.fakeRequest(global, requestTracker, stubTracker, paramParser);
 
     this.install = function() {
+      if (global.XMLHttpRequest === mockAjaxFunction) {
+        throw "MockAjax is already installed.";
+      }
+
       global.XMLHttpRequest = mockAjaxFunction;
     };
 

spec/integration/mock-ajax-spec.js

@@ -1,4 +1,31 @@
 describe("mockAjax", function() {
+  it("throws an error if installed multiple times", function() {
+    var fakeXmlHttpRequest = jasmine.createSpy('fakeXmlHttpRequest'),
+      fakeGlobal = { XMLHttpRequest: fakeXmlHttpRequest },
+      mockAjax = new window.MockAjax(fakeGlobal);
+
+    function doubleInstall() {
+      mockAjax.install();
+      mockAjax.install();
+    }
+
+    expect(doubleInstall).toThrow();
+  });
+
+  it("does not throw an error if uninstalled between installs", function() {
+    var fakeXmlHttpRequest = jasmine.createSpy('fakeXmlHttpRequest'),
+      fakeGlobal = { XMLHttpRequest: fakeXmlHttpRequest },
+      mockAjax = new window.MockAjax(fakeGlobal);
+
+    function sequentialInstalls() {
+      mockAjax.install();
+      mockAjax.uninstall();
+      mockAjax.install();
+    }
+
+    expect(sequentialInstalls).not.toThrow();
+  });
+
   it("does not replace XMLHttpRequest until it is installed", function() {
     var fakeXmlHttpRequest = jasmine.createSpy('fakeXmlHttpRequest'),
         fakeGlobal = { XMLHttpRequest: fakeXmlHttpRequest },

src/mockAjax.js

@@ -7,6 +7,10 @@ getJasmineRequireObj().MockAjax = function($ajax) {
       mockAjaxFunction = $ajax.fakeRequest(global, requestTracker, stubTracker, paramParser);
 
     this.install = function() {
+      if (global.XMLHttpRequest === mockAjaxFunction) {
+        throw "MockAjax is already installed.";
+      }
+
       global.XMLHttpRequest = mockAjaxFunction;
     };