Treat CLI args as paths, not URLs (#239)

Fixes #237.

Author: jathak

CHANGELOG.md

@@ -1,3 +1,9 @@
+## 1.7.3
+
+* Fixes a bug where path arguments on the command line were incorrectly treated
+  as URLs, resulting in errors finding paths containing certain special
+  characters.
+
 ## 1.7.2
 
 ### Module Migrator

lib/src/migrator.dart

@@ -88,7 +88,7 @@ abstract class Migrator extends Command<Map<Uri, String>> {
     ];
     for (var entrypoint in entrypoints) {
       var tuple =
-          importCache.import(Uri.parse(entrypoint), baseImporter: importer);
+          importCache.import(p.toUri(entrypoint), baseImporter: importer);
       if (tuple == null) {
         throw MigrationException("Could not find Sass file at '$entrypoint'.");
       }

pubspec.yaml

@@ -1,5 +1,5 @@
 name: sass_migrator
-version: 1.7.2
+version: 1.7.3
 description: A tool for running migrations on Sass files
 homepage: https://github.com/sass/migrator
 

test/cli_dart_test.dart

@@ -58,6 +58,14 @@ void main() {
     ]).validate();
   });
 
+  test("treats file arguments as paths, not urls", () async {
+    await d.file("#file.scss", "a {b: (1 / 2)}").create();
+
+    await (await runMigrator(["division", "#file.scss"])).shouldExit(0);
+
+    await d.file("#file.scss", 'a {b: (1 * 0.5)}').validate();
+  });
+
   test("allows non-glob file arguments containing glob syntax", () async {
     await d.dir('[dir]', [d.file("test.scss", "a {b: (1 / 2)}")]).create();