prefer-set-has: Ignore arrays only checking existence once

docs/rules/prefer-set-has.md

@@ -8,28 +8,25 @@ This rule is fixable.
 
 ```js
 const array = [1, 2, 3];
-
-function hasValue(value) {
-	return array.includes(value);
-}
+const hasValue = value => array.includes(value);
 ```
 
 ## Pass
 
 ```js
 const set = new Set([1, 2, 3]);
-
-function hasValue(value) {
-	return set.has(value);
-}
+const hasValue = value => set.has(value);
 ```
 
 ```js
+// This array is not only checking existence.
 const array = [1, 2];
-
-function hasValue(value) {
-	return array.includes(value);
-}
-
+const hasValue = value => array.includes(value);
 array.push(3);
 ```
+
+```js
+// This array is only checked once.
+const array = [1, 2, 3];
+const hasOne = array.includes(1);
+```

rules/prefer-set-has.js

@@ -107,6 +107,35 @@ const isIncludesCall = node => {
 	);
 };
 
+const multipleCallNodeTypes = new Set([
+	'ForOfStatement',
+	'ForStatement',
+	'WhileStatement',
+	'DoWhileStatement',
+	'FunctionDeclaration',
+	'FunctionExpression',
+	'ArrowFunctionExpression',
+	'ObjectMethod',
+	'ClassMethod'
+]);
+
+const isMultipleCall = (identifier, node) => {
+	const root = node.parent.parent.parent;
+	let {parent} = identifier.parent; // `.include()` callExpression
+	while (
+		parent &&
+		parent !== root
+	) {
+		if (multipleCallNodeTypes.has(parent.type)) {
+			return true;
+		}
+
+		parent = parent.parent;
+	}
+
+	return false;
+};
+
 const create = context => {
 	return {
 		[selector]: node => {
@@ -115,7 +144,14 @@ const create = context => {
 
 			if (
 				identifiers.length === 0 ||
-				identifiers.some(node => !isIncludesCall(node))
+				identifiers.some(identifier => !isIncludesCall(identifier))
+			) {
+				return;
+			}
+
+			if (
+				identifiers.length === 1 &&
+				identifiers.every(identifier => !isMultipleCall(identifier, node))
 			) {
 				return;
 			}