prefer-includes and prefer-array-index-of: Prevent crash on Vue files (#2645)

Author: fisker

rules/shared/simple-array-search-rule.js

@@ -85,8 +85,11 @@ export default function simpleArraySearchRule({method, replacement}) {
 
 			const callbackScope = scopeManager.acquire(callback);
 			if (
+				// Can't use scopeManager in vue template
+				// https://github.com/vuejs/vue-eslint-parser/issues/263
+				!callbackScope
 				// `parameter` is used somewhere else
-				findVariable(callbackScope, parameter).references.some(({identifier}) => identifier !== parameterInBinaryExpression)
+				|| findVariable(callbackScope, parameter).references.some(({identifier}) => identifier !== parameterInBinaryExpression)
 				|| isFunctionSelfUsedInside(callback, callbackScope)
 			) {
 				return;

test/shared/simple-array-search-rule-tests.js

@@ -1,4 +1,5 @@
 import outdent from 'outdent';
+import {parsers} from '../utils/test.js';
 
 function snapshotTests({method, replacement}) {
 	return {
@@ -80,6 +81,13 @@ function snapshotTests({method, replacement}) {
 
 			// Already valid case
 			`foo.${replacement}(0)`,
+
+			// Can't use scopeManager in vue template
+			// https://github.com/vuejs/vue-eslint-parser/issues/263
+			{
+				code: `<template><div v-if="values.${method}(x => x === 'foo')"></div></template>`,
+				languageOptions: {parser: parsers.vue},
+			},
 		],
 
 		invalid: [