Allow non-glob file path arguments containing glob syntax (#196)

Author: faazshift

CHANGELOG.md

@@ -1,3 +1,8 @@
+## 1.4.5
+
+* Glob syntax will no longer be resolved if a file with that literal name
+  exists.
+
 ## 1.4.4
 
 ### Division Migrator

lib/src/migrator.dart

@@ -81,8 +81,11 @@ abstract class Migrator extends Command<Map<Uri, String>> {
 
     var entrypoints = [
       for (var argument in argResults!.rest)
-        for (var entry in Glob(argument).listFileSystemSync(fileSystem))
-          if (entry is File) entry.path
+        if (File(argument).existsSync())
+          argument
+        else
+          for (var entry in Glob(argument).listFileSystemSync(fileSystem))
+            if (entry is File) entry.path
     ];
     for (var entrypoint in entrypoints) {
       var tuple =

pubspec.yaml

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

test/cli_dart_test.dart

@@ -64,6 +64,16 @@ void main() {
     ]).validate();
   });
 
+  test("allows non-glob file arguments containing glob syntax", () async {
+    await d.dir('[dir]', [d.file("test.scss", "a {b: (1 / 2)}")]).create();
+
+    await (await runMigrator(["division", "[dir]/test.scss"])).shouldExit(0);
+
+    await d.dir('[dir]', [
+      d.file("test.scss", '@use "sass:math";\n\na {b: math.div(1, 2)}')
+    ]).validate();
+  });
+
   group("with --dry-run", () {
     test("prints the name of a file that would be migrated", () async {
       await d.file("test.scss", "a {b: abs(-1)}").create();