Properly migrate built-ins with underscores (#165)

Fixes #138.

Author: jathak

CHANGELOG.md

@@ -1,3 +1,10 @@
+## 1.2.5
+
+### Module Migrator
+
+* The migrator now properly migrates built-in function calls with underscores
+  (e.g. `map_get`).
+
 ## 1.2.4
 
 ### Module Migrator

lib/src/migrators/module/references.dart

@@ -300,7 +300,7 @@ class _ReferenceVisitor extends RecursiveAstVisitor {
     for (var function in functions) {
       if (_isCssCompatibilityOverload(function)) continue;
       if (function.name.asPlain == null) continue;
-      var name = function.name.asPlain;
+      var name = function.name.asPlain.replaceAll('_', '-');
       var module = builtInFunctionModules[name];
       if (module != null) _sources[function] = BuiltInSource(module);
     }

pubspec.yaml

@@ -1,5 +1,5 @@
 name: sass_migrator
-version: 1.2.4
+version: 1.2.5
 description: A tool for running migrations on Sass files
 author: Jennifer Thakar <[email protected]>
 homepage: https://github.com/sass/migrator

test/migrators/module/built_in_functions/underscores.hrx

@@ -0,0 +1,12 @@
+<==> input/entrypoint.scss
+$a: map_get((a: 1), a);
+$b: str_length("hello");
+$c: scale_color(blue, $lightness: -10%);
+
+<==> output/entrypoint.scss
+@use "sass:color";
+@use "sass:map";
+@use "sass:string";
+$a: map.get((a: 1), a);
+$b: string.length("hello");
+$c: color.scale(blue, $lightness: -10%);