Merge pull request #22 from kwonoj/chore-lint

style(testscheduler): update code lint, hint friendly

Author: kwonoj

package.json

@@ -25,7 +25,7 @@
     "prepush": "npm-run-all build test",
     "commitmsg": "commitlint -e",
     "test": "jest --config jest.json --coverage",
-    "lint": "tslint \"src/**/*.ts\" \"spec/**/*.ts\"",
+    "lint": "tslint --type-check -c tslint.json -p tsconfig.json \"src/**/*.ts\" \"spec/**/*.ts\"",
     "lint:staged": "lint-staged",
     "build": "npm-run-all build:clean && tsc",
     "build:clean": "shx rm -rf ./dist",

src/scheduler/TestScheduler.ts

@@ -19,20 +19,6 @@ class TestScheduler extends VirtualTimeScheduler {
     super(VirtualAction, Number.POSITIVE_INFINITY);
   }
 
-  private materializeInnerObservable<T>(observable: Observable<any>, outerFrame: number): Array<TestMessage<T>> {
-    const innerObservableMetadata: Array<TestMessage<T>> = [];
-    const pushMetaData = (notification: Notification<T>) =>
-      innerObservableMetadata.push(new TestMessageValue<T>(this.frame - outerFrame, notification));
-
-    observable.subscribe(
-      value => pushMetaData(Notification.createNext(value)),
-      err => pushMetaData(Notification.createError(err)),
-      () => pushMetaData(Notification.createComplete())
-    );
-
-    return innerObservableMetadata;
-  }
-
   public getMarbles<T = string>(observable: Observable<T>, unsubscriptionMarbles: string | null = null) {
     if (this.autoFlush) {
       throw new Error('not implemented');
@@ -102,6 +88,20 @@ class TestScheduler extends VirtualTimeScheduler {
     this.hotObservables.push(subject);
     return subject;
   }
+
+  private materializeInnerObservable<T>(observable: Observable<any>, outerFrame: number): Array<TestMessage<T>> {
+    const innerObservableMetadata: Array<TestMessage<T>> = [];
+    const pushMetaData = (notification: Notification<T>) =>
+      innerObservableMetadata.push(new TestMessageValue<T>(this.frame - outerFrame, notification));
+
+    observable.subscribe(
+      value => pushMetaData(Notification.createNext(value)),
+      err => pushMetaData(Notification.createError(err)),
+      () => pushMetaData(Notification.createComplete())
+    );
+
+    return innerObservableMetadata;
+  }
 }
 
 export { TestScheduler };

tslint.json

@@ -11,6 +11,12 @@
       true,
       "spaces"
     ],
+    "member-ordering": [
+      true,
+      {
+        "order": "fields-first"
+      }
+    ],
     "max-line-length": [
       true,
       150