Skip to content

Commit 8ba1954

Browse files
committed
Merge pull request #219 from zertosh/jsx-namespaced
Add namespaced JSX support to jsx-no-undef
2 parents 67a6eea + e5e29ce commit 8ba1954

File tree

3 files changed

+18
-0
lines changed

3 files changed

+18
-0
lines changed

lib/rules/jsx-no-undef.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,9 @@ module.exports = function(context) {
6262
case 'JSXMemberExpression':
6363
node = node.name.object;
6464
break;
65+
case 'JSXNamespacedName':
66+
node = node.name.namespace;
67+
break;
6568
default:
6669
break;
6770
}

tests/lib/rules/jsx-no-undef.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,5 +45,12 @@ ruleTester.run('jsx-no-undef', rule, {
4545
message: '\'Appp\' is not defined.'
4646
}],
4747
ecmaFeatures: {jsx: true}
48+
},
49+
{
50+
code: '/*eslint no-undef:1*/ var React; React.render(<Apppp:Foo />);',
51+
errors: [{
52+
message: '\'Apppp\' is not defined.'
53+
}],
54+
ecmaFeatures: {jsx: true}
4855
}]
4956
});

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

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,14 @@ ruleTester.run('no-unused-vars', rule, {
7979
ecmaFeatures: {
8080
jsx: true
8181
}
82+
}, {
83+
code: '\
84+
/*eslint jsx-uses-vars:1*/\
85+
var App;\
86+
<App:Hello />',
87+
ecmaFeatures: {
88+
jsx: true
89+
}
8290
}, {
8391
code: '\
8492
/*eslint jsx-uses-vars:1*/\

0 commit comments

Comments
 (0)