Revert "Revert recent member-ordering changes (palantir#2118)"

This reverts commit aaa2319.

test/rules/member-ordering/custom-categories/test.ts.lint

@@ -0,0 +1,10 @@
+class C {
+    static foo() {}
+    protected static bar = 0;
+    static baz();
+
+    constructor();
+
+    static bang();
+    ~~~~~~~~~~~~~~ [Declaration of static non-private not allowed after declaration of constructor. Instead, this should come at the beginning of the class/interface.]
+}

test/rules/member-ordering/custom-categories/tslint.json

@@ -0,0 +1,18 @@
+{
+    "rules": {
+        "member-ordering": [true, {
+            "order": [
+                {
+                    "name": "static non-private",
+                    "kinds": [
+                        "public-static-field",
+                        "protected-static-field",
+                        "public-static-method",
+                        "protected-static-method"
+                    ]
+                },
+                "constructor"
+            ]
+        }]
+    }
+}

test/rules/member-ordering/omit-access-modifier/test.ts.lint

@@ -0,0 +1,12 @@
+class C {
+    private static x = 0;
+    static y = 1;
+
+    x = 0;
+    private y = 1;
+
+    constructor() {}
+
+    static z = 2;
+    ~~~~~~~~~~~~~ [Declaration of static field not allowed after declaration of constructor. Instead, this should come at the beginning of the class/interface.]
+}

test/rules/member-ordering/omit-access-modifier/tslint.json

@@ -0,0 +1,9 @@
+{
+    "rules": {
+        "member-ordering": [true, { "order": [
+            "static-field",
+            "instance-field",
+            "constructor"
+        ]}]
+    }
+}

test/rules/member-ordering/public-before-private/test.ts.lint

@@ -0,0 +1,10 @@
+class Foo {
+    private x: number;
+    private bar(): any {
+        var bla: { a: string } = {a: '1'};
+    }
+    y: number;
+    ~~~~~~~~~~ [0]
+}
+
+[0]: Declaration of public member not allowed after declaration of private member. Instead, this should come at the beginning of the class/interface.

test/rules/member-ordering/static-before-instance/test.ts.lint

@@ -0,0 +1,10 @@
+class Foo {
+    x: number;
+    static y: number;
+    ~~~~~~~~~~~~~~~~~ [0]
+    constructor() {
+        // nothing to do
+    }
+}
+
+[0]: Declaration of static member not allowed after declaration of instance member. Instead, this should come at the beginning of the class/interface.

test/rules/member-ordering/method/test.ts.lint → test/rules/member-ordering/variables-before-functions/test.ts.lint

@@ -52,5 +52,20 @@ class Constructor2 {
     ~~~~~~~~~~~~~~~~~ [0]
 }
 
-[0]: Declaration of public instance member variable not allowed to appear after declaration of public instance member function
+// Works for type literal, just like interface
+type T = {
+    x(): void;
+    y: number;
+    ~~~~~~~~~~ [0]
+}
+
+// Works for class inside object literal
+const o = {
+    foo: class C {
+        x(): void;
+        y: number;
+        ~~~~~~~~~~ [0]
+    }
+}
 
+[0]: Declaration of field not allowed after declaration of method. Instead, this should come at the beginning of the class/interface.