Merge pull request #1795 from lynxtaa/fix-sort-comp-statics

sort-comp: enforcing static lifecycle methods order

Author: ljharb

lib/rules/sort-comp.js

@@ -172,7 +172,7 @@ module.exports = {
         }
       }
 
-      if (method.static) {
+      if (indexes.length === 0 && method.static) {
         const staticIndex = methodsOrder.indexOf('static-methods');
         if (staticIndex >= 0) {
           indexes.push(staticIndex);

tests/lib/rules/sort-comp.js

@@ -504,6 +504,15 @@ ruleTester.run('sort-comp', rule, {
     ].join('\n'),
     parser: 'babel-eslint',
     errors: [{message: 'render should be placed after displayName'}]
+  }, {
+    // Must validate static lifecycle methods
+    code: [
+      'class Hello extends React.Component {',
+      '  static getDerivedStateFromProps() {}',
+      '  constructor() {}',
+      '}'
+    ].join('\n'),
+    errors: [{message: 'getDerivedStateFromProps should be placed after constructor'}]
   }, {
     // Type Annotations should not be at the top by default
     code: [