Do not allow to register itself with TraceKit twice

Author: TvoroG

src/raven.js

@@ -21,7 +21,8 @@ var _Raven = window.Raven,
         tags: {},
         extra: {}
     },
-    authQueryString;
+    authQueryString,
+    isRavenInstalled = false;
 
 /*
  * The core Raven singleton
@@ -121,8 +122,9 @@ var Raven = {
      * @return {Raven}
      */
     install: function() {
-        if (isSetup()) {
+        if (isSetup() && !isRavenInstalled) {
             TraceKit.report.subscribe(handleStackInfo);
+            isRavenInstalled = true;
         }
 
         return Raven;
@@ -216,6 +218,7 @@ var Raven = {
      */
     uninstall: function() {
         TraceKit.report.uninstall();
+        isRavenInstalled = false;
 
         return Raven;
     },

test/raven.test.js

@@ -1247,6 +1247,14 @@ describe('Raven (public API)', function() {
             assert.isTrue(TraceKit.report.subscribe.calledOnce);
             assert.equal(TraceKit.report.subscribe.lastCall.args[0], handleStackInfo);
         });
+
+        it('should not register itself more than once', function() {
+            this.sinon.stub(window, 'isSetup').returns(true);
+            this.sinon.stub(TraceKit.report, 'subscribe');
+            Raven.install();
+            Raven.install();
+            assert.isTrue(TraceKit.report.subscribe.calledOnce);
+        });
     });
 
     describe('.wrap', function() {
@@ -1392,6 +1400,13 @@ describe('Raven (public API)', function() {
             Raven.uninstall();
             assert.isTrue(TraceKit.report.uninstall.calledOnce);
         });
+
+        it('should set isRavenInstalled flag to false', function() {
+            isRavenInstalled = true;
+            this.sinon.stub(TraceKit.report, 'uninstall');
+            Raven.uninstall();
+            assert.isFalse(isRavenInstalled);
+        });
     });
 
     describe('.setUserContext', function() {