add rule coverage for every collection callback method

Author: Mathis Girault

eslint-plugin/lib/rules/avoid-getting-size-collection-in-loop.js

@@ -34,7 +34,7 @@ module.exports = {
   },
   create: function (context) {
     const SIZE_PROPERTIES = ["length", "size"]; // We only include static analysis on size and length properties at the moment
-    const CALLBACK_METHODS = ["map", "forEach", "filter", "some", "every"];
+    const CALLBACK_METHODS = ["filter", "find", "findIndex", "findLast", "findLastIndex", "some", "every", "flatMap", "forEach", "map", "reduce", "reduceRight", "some"];
 
     /**
      * Checks if a node is a .length or .size property access (dot or bracket notation).

eslint-plugin/tests/lib/rules/avoid-getting-size-collection-in-loop.js

@@ -182,30 +182,100 @@ ruleTester.run("avoid-getting-size-collection-in-loop", rule, {
       errors: [expectedError],
     },
 
-    // 5. In callback of array methods
+    // 5. In callback of collection methods
     {
       code: `
-        arr.forEach(item => {
+        arr.filter(item => {
           doSomething(arr.length);
         });
       `,
       errors: [expectedError],
     },
     {
       code: `
-        arr.map(function(x) {
-          return set.size;
+        arr.find(item => {
+          doSomething(arr.length);
         });
       `,
       errors: [expectedError],
     },
     {
       code: `
-        arr.filter(x => arr.size());
+        arr.findIndex(item => {
+          doSomething(arr.length);
+        });
+      `,
+      errors: [expectedError],
+    },
+    {
+      code: `
+        arr.findLast(item => {
+          doSomething(arr.length);
+        });
+      `,
+      errors: [expectedError],
+    },
+    {
+      code: `
+        arr.findLastIndex(item => {
+          doSomething(arr.length);
+        });
+      `,
+      errors: [expectedError],
+    },
+    {
+      code: `
+        arr.some(item => {
+          doSomething(arr.length);
+        });
+      `,
+      errors: [expectedError],
+    },
+    {
+      code: `
+        arr.every(item => {
+          doSomething(arr.length);
+        });
+      `,
+      errors: [expectedError],
+    },
+    {
+      code: `
+        arr.flatMap(item => {
+          doSomething(arr.length);
+        });
+      `,
+      errors: [expectedError],
+    },
+    {
+      code: `
+        arr.forEach(item => {
+          doSomething(arr.length);
+        });
+      `,
+      errors: [expectedError],
+    },{
+      code: `
+        arr.map(item => {
+          doSomething(arr.length);
+        });
+      `,
+      errors: [expectedError],
+    },{
+      code: `
+        arr.reduce(item => {
+          doSomething(arr.length);
+        });
+      `,
+      errors: [expectedError],
+    },{
+      code: `
+        arr.reduceRight(item => {
+          doSomething(arr.length);
+        });
       `,
       errors: [expectedError],
     },
-
     // 6. Multiple in one loop
     {
       code: `