fix: skip collection hooks

quavedev · Feb 20, 2024 · b716798 · b716798
1 parent 4772df3
commit b716798
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
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/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();