feat(rule): rxjs-no-deep-operators.

Author: cartant

README.md

@@ -74,11 +74,12 @@ The package includes the following rules (none of which are enabled by default):
 | `rxjs-finnish` | Enforces the use of [Finnish notation](https://medium.com/@benlesh/observables-and-finnish-notation-df8356ed1c9b). | None |
 | `rxjs-no-add` | Disallows the importation of patched observables and operators. | See below |
 | `rxjs-no-create` | Disallows the calling of `Observable.create`. Useful as a warning. | None |
+| `rxjs-no-deep-operators` | Disallows deep importation from 'rxjs/operators'. Deep imports won't be in available in RxJS v6. | None |
 | `rxjs-no-do` | I do without `do` operators. [Do you not?](https://youtu.be/spG-Yj0zEyc) Well, `do` isn't always a code smell, but this rule can be useful as a warning. | None |
 | `rxjs-no-finnish` | Disallows the use of [Finnish notation](https://medium.com/@benlesh/observables-and-finnish-notation-df8356ed1c9b). | None |
 | `rxjs-no-ignored-error` | Disallows the calling of `subscribe` without specifying an error handler. | None |
 | `rxjs-no-ignored-subscribe` | Disallows the calling of subscribe without specifying arguments. | None |
-| `rxjs-no-operator` | Disallows importation from the `operator` directory. Useful if you prefer ['lettable' operators](https://github.com/ReactiveX/rxjs/blob/master/doc/lettable-operators.md) - which are located in the `operators` directory. | None |
+| `rxjs-no-operator` | Disallows importation from 'rxjs/operator'. Useful if you prefer ['lettable' operators](https://github.com/ReactiveX/rxjs/blob/master/doc/lettable-operators.md) - which are located in the `operators` directory. | None |
 | `rxjs-no-patched` | Disallows the calling of patched methods. Methods must be imported and called explicitly - not via `Observable` or `Observable.prototype`. | See below |
 | `rxjs-no-subject-unsubscribe` | Disallows calling the `unsubscribe` method of a `Subject` instance. For an explanation of why this can be a problem, see [this](https://stackoverflow.com/a/45112125/6680611) Stack Overflow answer. | None |
 | `rxjs-no-subject-value` | Disallows accessing the `value` property of a `BehaviorSubject` instance. | None |

docs/index.md

@@ -14,11 +14,12 @@ The package includes the following rules (none of which are enabled by default):
 | `rxjs-finnish` | Enforces the use of [Finnish notation](https://medium.com/@benlesh/observables-and-finnish-notation-df8356ed1c9b). | None |
 | `rxjs-no-add` | Disallows the importation of patched observables and operators. | See below |
 | `rxjs-no-create` | Disallows the calling of `Observable.create`. Useful as a warning. | None |
+| `rxjs-no-deep-operators` | Disallows deep importation from 'rxjs/operators'. Deep imports won't be in available in RxJS v6. | None |
 | `rxjs-no-do` | I do without `do` operators. [Do you not?](https://youtu.be/spG-Yj0zEyc) Well, `do` isn't always a code smell, but this rule can be useful as a warning. | None |
 | `rxjs-no-finnish` | Disallows the use of [Finnish notation](https://medium.com/@benlesh/observables-and-finnish-notation-df8356ed1c9b). | None |
 | `rxjs-no-ignored-error` | Disallows the calling of `subscribe` without specifying an error handler. | None |
 | `rxjs-no-ignored-subscribe` | Disallows the calling of subscribe without specifying arguments. | None |
-| `rxjs-no-operator` | Disallows importation from the `operator` directory. Useful if you prefer ['lettable' operators](https://github.com/ReactiveX/rxjs/blob/master/doc/lettable-operators.md) - which are located in the `operators` directory. | None |
+| `rxjs-no-operator` | Disallows importation from 'rxjs/operator'. Useful if you prefer ['lettable' operators](https://github.com/ReactiveX/rxjs/blob/master/doc/lettable-operators.md) - which are located in the `operators` directory. | None |
 | `rxjs-no-patched` | Disallows the calling of patched methods. Methods must be imported and called explicitly - not via `Observable` or `Observable.prototype`. | See below |
 | `rxjs-no-subject-unsubscribe` | Disallows calling the `unsubscribe` method of a `Subject` instance. For an explanation of why this can be a problem, see [this](https://stackoverflow.com/a/45112125/6680611) Stack Overflow answer. | None |
 | `rxjs-no-subject-value` | Disallows accessing the `value` property of a `BehaviorSubject` instance. | None |

fixtures/no-deep-operators/fixture.ts

@@ -0,0 +1,3 @@
+import { Observable } from "rxjs";
+import { map } from "rxjs/operators/map";
+import { tap } from "rxjs/operators";

fixtures/no-deep-operators/tsconfig.json

@@ -0,0 +1,13 @@
+{
+  "compilerOptions": {
+    "baseUrl": ".",
+    "lib": ["es2015"],
+    "noEmit": true,
+    "paths": {
+      "rxjs": ["../node_modules/rxjs"]
+    },
+    "skipLibCheck": true,
+    "target": "es5"
+  },
+  "include": ["fixture.ts"]
+}

fixtures/no-deep-operators/tslint.json

@@ -0,0 +1,8 @@
+{
+  "defaultSeverity": "error",
+  "jsRules": {},
+  "rules": {
+    "rxjs-no-deep-operators": { "severity": "error" }
+  },
+  "rulesDirectory": "../../build/rules"
+}

source/fixtures-spec.ts

@@ -181,6 +181,17 @@ describe("fixtures", function (): void {
             });
         });
 
+        describe("no-deep-operators", () => {
+
+            it("should effect 'rxjs-no-deep-operators' errors", () => {
+
+                const result = lint("no-deep-operators", "tslint.json");
+
+                expect(result).to.have.property("errorCount", 1);
+                result.failures.forEach(failure => expect(failure).to.have.property("ruleName", "rxjs-no-deep-operators"));
+            });
+        });
+
         describe("no-do", () => {
 
             it("should effect 'rxjs-no-do' errors", () => {

source/rules/rxjsNoDeepOperatorsRule.ts

@@ -0,0 +1,41 @@
+/**
+ * @license Use of this source code is governed by an MIT-style license that
+ * can be found in the LICENSE file at https://github.com/cartant/rxjs-tslint-rules
+ */
+/*tslint:disable:no-use-before-declare*/
+
+import * as Lint from "tslint";
+import * as ts from "typescript";
+
+export class Rule extends Lint.Rules.AbstractRule {
+
+    public static metadata: Lint.IRuleMetadata = {
+        description: "Disallows deep importation from 'rxjs/operators'.",
+        options: null,
+        optionsDescription: "Not configurable.",
+        requiresTypeInfo: false,
+        ruleName: "rxjs-no-deep-operators",
+        type: "functionality",
+        typescriptOnly: false
+    };
+
+    public static FAILURE_STRING = "Deep importation from 'rxjs/operators' is forbidden";
+
+    public apply(sourceFile: ts.SourceFile): Lint.RuleFailure[] {
+        return this.applyWithWalker(new Walker(sourceFile, this.getOptions()));
+    }
+}
+
+class Walker extends Lint.RuleWalker {
+
+    public visitImportDeclaration(node: ts.ImportDeclaration): void {
+
+        const moduleSpecifier = node.moduleSpecifier.getText();
+
+        if (/^['"]rxjs\/operators\/\w+/.test(moduleSpecifier)) {
+            this.addFailureAtNode(node, Rule.FAILURE_STRING);
+        }
+
+        super.visitImportDeclaration(node);
+    }
+}

source/rules/rxjsNoOperatorRule.ts

@@ -10,7 +10,7 @@ import * as ts from "typescript";
 export class Rule extends Lint.Rules.AbstractRule {
 
     public static metadata: Lint.IRuleMetadata = {
-        description: "Disallows importation from the 'operator' directory.",
+        description: "Disallows importation from 'rxjs/operator'.",
         options: null,
         optionsDescription: "Not configurable.",
         requiresTypeInfo: false,
@@ -19,7 +19,7 @@ export class Rule extends Lint.Rules.AbstractRule {
         typescriptOnly: false
     };
 
-    public static FAILURE_STRING = "RxJS importation from the 'operator' directory is forbidden";
+    public static FAILURE_STRING = "Importation from 'rxjs/operator' is forbidden";
 
     public apply(sourceFile: ts.SourceFile): Lint.RuleFailure[] {
         return this.applyWithWalker(new Walker(sourceFile, this.getOptions()));