no-unused-array-method-return: Cover computed method names

Fixes #3549

Author: sindresorhus

rules/no-unused-array-method-return.js

@@ -98,7 +98,8 @@ function getVariableValue(node, context) {
 
 	// Supported variable inference boundary:
 	// - exactly one binding definition
-	// - unannotated plain parameters and explicitly array-typed plain parameters or variables
+	// - unannotated or explicitly array-typed plain parameters
+	// - explicitly array-typed variables
 	// - a `VariableDeclarator` whose id is the same identifier we are resolving
 	// - the original declarator initializer for unannotated variables only
 	//

test/no-unused-array-method-return.js

@@ -5,7 +5,6 @@ const {test} = getTester(import.meta);
 
 test.snapshot({
 	valid: [
-		'new array.map(fn);',
 		'map(fn);',
 		'array.notMap(fn);',
 		'array[map](fn);',
@@ -279,6 +278,7 @@ test.snapshot({
 		'const assertion = expectation(value); assertion.includes(expected);',
 		'expect.custom(value).includes(expected);',
 		'function Symbol(value) { return value; } Symbol([]).slice(1);',
+		'const method = "map"; array[method](fn);',
 	],
 });
 

test/snapshots/no-unused-array-method-return.js.md

@@ -881,3 +881,19 @@ Generated by [AVA](https://avajs.dev).
     > 1 | function Symbol(value) { return value; } Symbol([]).slice(1);␊
         |                                                     ^^^^^ Do not ignore the return value of \`.slice(…)\`.␊
     `
+
+## invalid(54): const method = "map"; array[method](fn);
+
+> Input
+
+    `␊
+      1 | const method = "map"; array[method](fn);␊
+    `
+
+> Error 1/1
+
+    `␊
+    Message:␊
+    > 1 | const method = "map"; array[method](fn);␊
+        |                             ^^^^^^ Do not ignore the return value of \`.map(…)\`.␊
+    `