Simplify approvals store initialization.

app/assets/javascripts/merge_request_widget/approvals/approvals_store.js.es6

@@ -4,10 +4,10 @@
 (() => {
   let singleton;
 
-  class ApprovalsStore {
+  class MergeRequestApprovalsStore {
     constructor(rootStore) {
       if (!singleton) {
-        singleton = gl.ApprovalsStore = this;
+        singleton = this;
         this.init(rootStore);
       }
       return singleton;
@@ -63,6 +63,6 @@
       return this.rootStore.assignToData(key, data);
     }
   }
-  gl.ApprovalsStore = ApprovalsStore;
-})();
+  gl.MergeRequestApprovalsStore = MergeRequestApprovalsStore;
+})(window.gl || (window.gl = {}));
 

app/assets/javascripts/merge_request_widget/widget_store.js.es6

@@ -28,7 +28,7 @@
     }
 
     initApprovals() {
-      gl.ApprovalsStore = new gl.ApprovalsStore(this);
+      gl.ApprovalsStore = new gl.MergeRequestApprovalsStore(this);
       this.assignToData('approvals', {});
     }
 

spec/javascripts/approvals/approvals_store_spec.js.es6

@@ -8,9 +8,6 @@ $.rails = {
 };
 
 (() => {
-  // store constructor reference, to overwrite ApprovalStore singleton
-  gl.ApprovalsConstructor = gl.ApprovalsStore;
-
   // stand in for promise returned by api calls
   const mockThenable = {
     then() {
@@ -34,7 +31,7 @@ $.rails = {
   describe('Approvals Store', function () {
     beforeEach(function () {
       this.rootStore = mockRootStore;
-      this.approvalsStore = new gl.ApprovalsConstructor(this.rootStore);
+      this.approvalsStore = new gl.MergeRequestApprovalsStore(this.rootStore);
     });
 
     it('should define all needed approval api calls', function () {