Merge pull request #333 from lencioni/pascal-caps

Allow consecutive uppercase letters in jsx-pascal-case (fixes #328)

Author: yannickcr

docs/rules/jsx-pascal-case.md

@@ -20,6 +20,10 @@ The following patterns are considered warnings:
 <test_component />
 ```
 
+```js
+<YELLING />
+```
+
 The following patterns are not considered warnings:
 
 ```js
@@ -32,6 +36,10 @@ The following patterns are not considered warnings:
 </TestComponent>
 ```
 
+```js
+<CSSTransitionGroup />
+```
+
 ## When Not To Use It
 
-If you are not using JSX.
+If you are not using JSX.

lib/rules/jsx-pascal-case.js

@@ -11,7 +11,7 @@ var variableUtil = require('../util/variable');
 // Constants
 // ------------------------------------------------------------------------------
 
-var PASCAL_CASE_REGEX = /^[A-Z][a-z]+(?:[A-Z][a-z]+)*$/;
+var PASCAL_CASE_REGEX = /^[A-Z]+[a-z]+(?:[A-Z]+[a-z]*)*$/;
 
 // ------------------------------------------------------------------------------
 // Rule Definition

tests/lib/rules/jsx-pascal-case.js

@@ -25,6 +25,22 @@ ruleTester.run('jsx-pascal-case', rule, {
     ecmaFeatures: {
       jsx: true
     }
+  }, {
+    code: [
+      'var CSSTransitionGroup;',
+      '<CSSTransitionGroup />'
+    ].join('\n'),
+    ecmaFeatures: {
+      jsx: true
+    }
+  }, {
+    code: [
+      'var BetterThanCSS;',
+      '<BetterThanCSS />'
+    ].join('\n'),
+    ecmaFeatures: {
+      jsx: true
+    }
   }, {
     code: [
       'var TestComponent;',
@@ -55,6 +71,15 @@ ruleTester.run('jsx-pascal-case', rule, {
       jsx: true
     },
     errors: [{message: 'Imported JSX component test_component must be in PascalCase'}]
+  }, {
+    code: [
+      'var YELLING;',
+      '<YELLING />'
+    ].join('\n'),
+    ecmaFeatures: {
+      jsx: true
+    },
+    errors: [{message: 'Imported JSX component YELLING must be in PascalCase'}]
   }, {
     code: [
       'var testComponent;',