fix: Don't ban root-imported operators.

Author: cartant

source/rules/ban-operators.ts

@@ -64,7 +64,7 @@ const rule = ruleCreator({
     }
 
     return {
-      [String.raw`ImportDeclaration[source.value=/^rxjs(\u002foperators)?$/] > ImportSpecifier`]:
+      [String.raw`ImportDeclaration[source.value=/^rxjs\u002foperators$/] > ImportSpecifier`]:
         (node: es.ImportSpecifier) => {
           const identifier = node.imported;
           const failure = getFailure(identifier.name);

tests/rules/ban-operators.ts

@@ -16,6 +16,19 @@ ruleTester({ types: false }).run("ban-operators", rule, {
     {
       code: `import { concat, merge as m, mergeMap as mm } from "rxjs";`,
     },
+    {
+      // This won't effect errors, because only imports from "rxjs/operators"
+      // are checked. To support banning operators from "rxjs", it'll need to
+      // check types.
+      code: `import { concat, merge as m, mergeMap as mm } from "rxjs";`,
+      options: [
+        {
+          concat: true,
+          merge: "because I say so",
+          mergeMap: false,
+        },
+      ],
+    },
   ],
   invalid: [
     fromFixture(
@@ -34,21 +47,5 @@ ruleTester({ types: false }).run("ban-operators", rule, {
         ],
       }
     ),
-    fromFixture(
-      stripIndent`
-        import { concat, merge as m, mergeMap as mm } from "rxjs";
-                 ~~~~~~ [forbidden { "name": "concat", "explanation": "" }]
-                         ~~~~~ [forbidden { "name": "merge", "explanation": ": because I say so" }]
-      `,
-      {
-        options: [
-          {
-            concat: true,
-            merge: "because I say so",
-            mergeMap: false,
-          },
-        ],
-      }
-    ),
   ],
 });