no-array-callback-reference: Fix optional chaining expressions being ignored (#2694)

Author: k-yle

rules/no-array-callback-reference.js

@@ -234,7 +234,6 @@ const create = context => ({
 				minimumArguments: 1,
 				maximumArguments: 2,
 				optionalCall: false,
-				optionalMember: false,
 				computed: false,
 			})
 			|| callExpression.callee.property.type !== 'Identifier'

test/no-array-callback-reference.js

@@ -57,6 +57,10 @@ test({
 		...simpleMethods.map(method => `foo.${method}(element => fn(element))`),
 		...reduceLikeMethods.map(method => `foo.${method}((accumulator, element) => fn(element))`),
 
+		// Optional chaining
+		...simpleMethods.map(method => `foo?.${method}(element => fn(element))`),
+		...reduceLikeMethods.map(method => `foo?.${method}((accumulator, element) => fn(element))`),
+
 		// `this.{map, filter, …}`
 		...simpleMethods.map(method => `this.${method}(fn)`),
 		...reduceLikeMethods.map(method => `this.${method}(fn)`),
@@ -160,6 +164,18 @@ test({
 				],
 			}),
 		),
+		...simpleMethodsExceptForEach.map(
+			method => invalidTestCase({
+				code: `foo?.${method}(fn)`,
+				method,
+				name: 'fn',
+				suggestions: [
+					`foo?.${method}((element) => fn(element))`,
+					`foo?.${method}((element, index) => fn(element, index))`,
+					`foo?.${method}((element, index, array) => fn(element, index, array))`,
+				],
+			}),
+		),
 		invalidTestCase({
 			code: 'foo.forEach(fn)',
 			method: 'forEach',