Skip to content

Commit e621e30

Browse files
mdjermanovicljharb
authored andcommitted
add tests/helpers/getRuleDefiner.js
1 parent 0effd62 commit e621e30

File tree

3 files changed

+15
-4
lines changed

3 files changed

+15
-4
lines changed

tests/helpers/getRuleDefiner.js

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
'use strict';
2+
3+
const eslint = require('eslint');
4+
5+
// `ruleTester` is a RuleTester instance
6+
const getRuleDefiner = (ruleTester) => (typeof Symbol !== 'undefined' && Symbol.for && ruleTester[Symbol.for('react.RuleTester.RuleDefiner')])
7+
|| ruleTester.linter
8+
|| eslint.linter
9+
|| eslint.Linter;
10+
11+
module.exports = getRuleDefiner;

tests/lib/rules/jsx-uses-react.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,10 @@
99
// Requirements
1010
// -----------------------------------------------------------------------------
1111

12-
const eslint = require('eslint');
1312
const rule = require('../../helpers/getESLintCoreRule')('no-unused-vars');
1413

1514
const RuleTester = require('../../helpers/ruleTester');
15+
const getRuleDefiner = require('../../helpers/getRuleDefiner');
1616

1717
const parsers = require('../../helpers/parsers');
1818

@@ -35,7 +35,7 @@ const settings = {
3535
// -----------------------------------------------------------------------------
3636

3737
const ruleTester = new RuleTester({ parserOptions });
38-
const ruleDefiner = ruleTester[Symbol.for('react.RuleTester.RuleDefiner')] || ruleTester.linter || eslint.linter || eslint.Linter;
38+
const ruleDefiner = getRuleDefiner(ruleTester);
3939
ruleDefiner.defineRule('react/jsx-uses-react', require('../../../lib/rules/jsx-uses-react'));
4040

4141
ruleTester.run('no-unused-vars', rule, {

tests/lib/rules/jsx-uses-vars.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,11 @@
99
// Requirements
1010
// -----------------------------------------------------------------------------
1111

12-
const eslint = require('eslint');
1312
const ruleNoUnusedVars = require('../../helpers/getESLintCoreRule')('no-unused-vars');
1413
const rulePreferConst = require('../../helpers/getESLintCoreRule')('prefer-const');
1514

1615
const RuleTester = require('../../helpers/ruleTester');
16+
const getRuleDefiner = require('../../helpers/getRuleDefiner');
1717

1818
const parsers = require('../../helpers/parsers');
1919

@@ -30,7 +30,7 @@ const parserOptions = {
3030
// -----------------------------------------------------------------------------
3131

3232
const ruleTester = new RuleTester({ parserOptions });
33-
const ruleDefiner = ruleTester[Symbol.for('react.RuleTester.RuleDefiner')] || ruleTester.linter || eslint.linter || eslint.Linter;
33+
const ruleDefiner = getRuleDefiner(ruleTester);
3434
ruleDefiner.defineRule('react/jsx-uses-vars', require('../../../lib/rules/jsx-uses-vars'));
3535

3636
ruleTester.run('no-unused-vars', ruleNoUnusedVars, {

0 commit comments

Comments
 (0)