[Fix] jsx-handler-names: handle whitespace

Fixes #2785
jsx-eslint · Sep 6, 2020 · a2e7a39 · a2e7a39
1 parent de2ec66
commit a2e7a39
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/lib/rules/jsx-handler-names.js b/lib/rules/jsx-handler-names.js
@@ -121,6 +121,7 @@ module.exports = {
         const expression = node.value.expression;
         const propValue = context.getSourceCode()
           .getText(checkInlineFunction && isInlineHandler(node) ? expression.body.callee : expression)
+          .replace(/\s*/g, '')
           .replace(/^this\.|.*::/, '');
 
         if (propKey === 'ref') {

diff --git a/tests/lib/rules/jsx-handler-names.js b/tests/lib/rules/jsx-handler-names.js
@@ -32,6 +32,20 @@ ruleTester.run('jsx-handler-names', rule, {
     code: '<TestComponent onChange={this.handleChange} />'
   }, {
     code: '<TestComponent onChange={this.props.onChange} />'
+  },
+  {
+    code: `<TestComponent
+              onChange={
+                this
+                  .handleChange
+              } />`
+  }, {
+    code: `<TestComponent
+              onChange={
+                this
+                  .props
+                  .handleChange
+              } />`
   }, {
     code: '<TestComponent onChange={handleChange} />',
     options: [{