Only ignore comments when finding insertion point (#197)

Author: jathak

CHANGELOG.md

@@ -1,3 +1,10 @@
+## 1.4.4
+
+### Division Migrator
+
+* Fix a bug where `@use "sass:math"` would sometimes be incorrectly inserted
+  after other rules.
+
 ## 1.4.3
 
 ### Division Migrator

lib/src/migrators/division.dart

@@ -91,10 +91,9 @@ class _DivisionMigrationVisitor extends MigrationVisitor {
     var useRules = _useRulesToInsert.join('\n');
     var insertionPoint = node.span.start;
     for (var child in node.children) {
-      if (child is UseRule || child is ForwardRule || child is ImportRule) {
-        insertionPoint = child.span.start;
-        break;
-      }
+      if (child is LoudComment || child is SilentComment) continue;
+      insertionPoint = child.span.start;
+      break;
     }
     addPatch(Patch.insert(insertionPoint, '$useRules\n\n'));
   }

pubspec.yaml

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

test/migrators/division/insert_before_style_rules.hrx

@@ -0,0 +1,22 @@
+<==> input/entrypoint.scss
+// Comment
+
+a {
+  b: 3 / $x;
+  c: (six / three);
+}
+
+@import "other";
+
+<==> output/entrypoint.scss
+// Comment
+
+@use "sass:math";
+@use "sass:list";
+
+a {
+  b: math.div(3, $x);
+  c: list.slash(six, three);
+}
+
+@import "other";