Skip to content

Commit fadd015

Browse files
committed
refactor
1 parent 1df040a commit fadd015

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

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

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -189,9 +189,7 @@ class ScopeManager {
189189
* @returns {void}
190190
*/
191191
addGlobals(names) {
192-
const globalScope = this.scopes[0];
193-
194-
globalScope.__addVariables(names);
192+
this.globalScope.__addVariables(names);
195193
}
196194

197195
attach() { } // eslint-disable-line class-methods-use-this -- Desired as instance method

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ describe("ScopeManager#addGlobals", () => {
2222
expect(globalScope.type).to.be.equal("global");
2323

2424
expect(globalScope.variables).to.have.length(0);
25+
expect(globalScope.set.size).to.be.equal(0);
2526
expect(globalScope.references).to.have.length(3);
2627
expect(globalScope.references[0].identifier.name).to.be.equal("foo");
2728
expect(globalScope.references[0].resolved).to.be.null;

0 commit comments

Comments
 (0)