fix: from-map rule is not working on TS files

package.json

@@ -16,13 +16,15 @@
     "@ava/babel": "^1.0.1",
     "@freaktechnik/eslint-config-node": "^7.1.0",
     "@freaktechnik/eslint-config-test": "^7.1.0",
+    "@typescript-eslint/parser": "^2.25.0",
     "ava": "^3.5.0",
     "codecov": "^3.6.5",
     "eclint": "^2.8.1",
     "eslint": "^6.8.0",
     "eslint-ava-rule-tester": "^4.0.0",
     "eslint-plugin-eslint-plugin": "^2.2.1",
-    "nyc": "^15.0.0"
+    "nyc": "^15.0.0",
+    "typescript": "^3.8.3"
   },
   "peerDependencies": {
     "eslint": ">=3.0.0"

rules/from-map.js

@@ -76,18 +76,18 @@ module.exports = {
                             }
                             // The original map callback from Array.from gets nested as a parameter to the callback from map.
                             const lastCallback = getCallback(mapCallback, mapThisArgument, `${firstCallback}${restParameterString}`),
-                                restParameters = sourceCode.getText().slice(callback.end, parent.end);
+                                restParameters = sourceCode.getText().slice(callback.loc.end.column, parent.loc.end.column);
                             return fixer.replaceTextRange([
-                                callback.start,
-                                node.end
+                                callback.loc.start.column,
+                                node.loc.end.column
                             ], `${functionStart}${lastCallback}${functionEnd}${restParameters}`);
                         }
 
                         // Move the map arguments to from.
                         const [ firstArgument ] = node.arguments;
                         return fixer.replaceTextRange([
-                            parent.end - FUNCTION_END.length,
-                            firstArgument.start
+                            parent.loc.end.column - FUNCTION_END.length,
+                            firstArgument.loc.start.column
                         ], PARAM_SEPARATOR);
                     }
                 });

test/rules/from-map-ts.js

@@ -0,0 +1,75 @@
+import AvaRuleTester from "eslint-ava-rule-tester";
+import test from "ava";
+import rule from "../../rules/from-map";
+
+const ruleTester = new AvaRuleTester(test, {
+    parser: require.resolve('@typescript-eslint/parser'),
+    parserOptions: {
+        ecmaVersion: 2020,
+        sourceType: 'module'
+    }
+});
+
+ruleTester.run('from-map', rule, {
+    valid: [
+        'array.map((t) => t.id)',
+        'Array.from(iterable).some((t) => t !== "a")',
+        'Array.from(iterable, (t) => t.id)'
+    ],
+    invalid: [
+        {
+            code: 'Array.from(iterable).map((t) => t.id)',
+            errors: [ {
+                message: 'Use mapFn callback of Array.from instead of map()',
+                column: 1,
+                line: 1
+            } ],
+            output: 'Array.from(iterable, (t) => t.id)'
+        },
+        {
+            code: 'Array.from(iterable, (t) => t.id, a).map((t) => t[0], b)',
+            errors: [ {
+                message: 'Use mapFn callback of Array.from instead of map()',
+                column: 1,
+                line: 1
+            } ],
+            output: 'Array.from(iterable, (t) => ((t) => t[0])(((t) => t.id)(t)), a)'
+        },
+        {
+            code: 'Array.from(iterable, function(t) { return t.id; }, a).map((t) => t[0])',
+            errors: [ {
+                message: 'Use mapFn callback of Array.from instead of map()',
+                column: 1,
+                line: 1
+            } ],
+            output: 'Array.from(iterable, function(t) { return ((t) => t[0])((function(t) { return t.id; }).call(this, t)); }, a)'
+        },
+        {
+            code: 'Array.from(iterable, function(t) { return t.id; }, a).map(function(t) { return t[0]; }, b)',
+            errors: [ {
+                message: 'Use mapFn callback of Array.from instead of map()',
+                column: 1,
+                line: 1
+            } ],
+            output: 'Array.from(iterable, function(t) { return (function(t) { return t[0]; }).call(b, (function(t) { return t.id; }).call(this, t)); }, a)'
+        },
+        {
+            code: 'Array.from(iterable, function(u) { return u.id; }, a).map(function(t, i) { return t[0]; }, b)',
+            errors: [ {
+                message: 'Use mapFn callback of Array.from instead of map()',
+                column: 1,
+                line: 1
+            } ],
+            output: 'Array.from(iterable, function(t, i) { return (function(t, i) { return t[0]; }).call(b, (function(u) { return u.id; }).call(this, t, i), i); }, a)'
+        },
+        {
+            code: 'Array.from(iterable, function(u, i) { return u.id; }, a).map(function(t) { return t[0]; }, b)',
+            errors: [ {
+                message: 'Use mapFn callback of Array.from instead of map()',
+                column: 1,
+                line: 1
+            } ],
+            output: 'Array.from(iterable, function(u, i) { return (function(t) { return t[0]; }).call(b, (function(u, i) { return u.id; }).call(this, u, i), i); }, a)'
+        }
+    ]
+});