refactor

Author: mdjermanovic

packages/eslint-scope/lib/scope-manager.js

@@ -189,9 +189,7 @@ class ScopeManager {
      * @returns {void}
      */
     addGlobals(names) {
-        const globalScope = this.scopes[0];
-
-        globalScope.__addVariables(names);
+        this.globalScope.__addVariables(names);
     }
 
     attach() { } // eslint-disable-line class-methods-use-this -- Desired as instance method

packages/eslint-scope/tests/add-globals.js

@@ -22,6 +22,7 @@ describe("ScopeManager#addGlobals", () => {
         expect(globalScope.type).to.be.equal("global");
 
         expect(globalScope.variables).to.have.length(0);
+        expect(globalScope.set.size).to.be.equal(0);
         expect(globalScope.references).to.have.length(3);
         expect(globalScope.references[0].identifier.name).to.be.equal("foo");
         expect(globalScope.references[0].resolved).to.be.null;