Merge pull request #6 from quavedev/5-issue-linting-meteor-collection…

…-hooks #5 issue linting meteor collection hooks
quavedev · Feb 20, 2024 · 05667aa · 05667aa
2 parents 4772df3 + ec288e6
commit 05667aa
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/lib/rules/no-sync-mongo-methods-on-server.js b/lib/rules/no-sync-mongo-methods-on-server.js
@@ -134,6 +134,14 @@ module.exports = {
                 return;
               }
             }
+            if(node.object.type === 'MemberExpression'){
+              // we can ignore longer than 1 call chain
+              debug(
+                `Skipping ${invalidFunction} to be considered error because it was used in a longer than 1 call chain`,
+                { object: node.object }
+              );
+              return;
+            }
             createError({
               context,
               node,

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@quave/eslint-plugin-meteor-quave",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "description": "Quave linting rules for ESLint",
   "main": "lib/index.js",
   "scripts": {

diff --git a/tests/lib/rules/no-sync-mongo-methods-on-server.js b/tests/lib/rules/no-sync-mongo-methods-on-server.js
@@ -20,6 +20,7 @@ ruleTester.run('no-sync-mongo-methods-on-server', rule, {
   only: true,
   valid: [
     { code: 'TestCollection.findOneAsync()' },
+    { code: 'UserAccess.after.insert((userId, doc) => {})' },
     {
       code: `
       var modulesCursor = ModulesCollection.find();