Expose CompileResult publicly, along with methods that return it

This also exposes CompileResult.includedUrls. Closes #1355 Partially addresses sass/embedded-protocol#62
sass · Jul 14, 2021 · 558ca0a · 558ca0a
1 parent 43b69e6
commit 558ca0a
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 15 deletions.
diff --git a/test/cli/shared/source_maps.dart b/test/cli/shared/source_maps.dart
@@ -34,7 +34,7 @@ void sharedTests(Future<TestProcess> runSass(Iterable<String> arguments)) {
     });
 
     test("contains mappings", () {
-      var result = sass.compileStringToResult("a {b: 1 + 2}");
+      var result = sass.compileStringToResult("a {b: 1 + 2}", sourceMap: true);
       expect(map,
           containsPair("mappings", result.sourceMap!.toJson()["mappings"]));
     });
@@ -287,7 +287,8 @@ void sharedTests(Future<TestProcess> runSass(Iterable<String> arguments)) {
       });
 
       test("contains mappings in the generated CSS", () {
-        var result = sass.compileStringToResult("a {b: 1 + 2}");
+        var result =
+            sass.compileStringToResult("a {b: 1 + 2}", sourceMap: true);
         expect(map,
             containsPair("mappings", result.sourceMap!.toJson()["mappings"]));
       });

diff --git a/test/dart_api/importer_test.dart b/test/dart_api/importer_test.dart
@@ -114,24 +114,28 @@ void main() {
   });
 
   test("uses an importer's source map URL", () {
-    var result = compileStringToResult('@import "orange";', importers: [
-      TestImporter((url) => Uri.parse("u:$url"), (url) {
-        var color = url.path;
-        return ImporterResult('.$color {color: $color}',
-            sourceMapUrl: Uri.parse("u:blue"), indented: false);
-      })
-    ]);
+    var result = compileStringToResult('@import "orange";',
+        importers: [
+          TestImporter((url) => Uri.parse("u:$url"), (url) {
+            var color = url.path;
+            return ImporterResult('.$color {color: $color}',
+                sourceMapUrl: Uri.parse("u:blue"), indented: false);
+          })
+        ],
+        sourceMap: true);
 
     expect(result.sourceMap!.urls, contains("u:blue"));
   });
 
   test("uses a data: source map URL if the importer doesn't provide one", () {
-    var result = compileStringToResult('@import "orange";', importers: [
-      TestImporter((url) => Uri.parse("u:$url"), (url) {
-        var color = url.path;
-        return ImporterResult('.$color {color: $color}', indented: false);
-      })
-    ]);
+    var result = compileStringToResult('@import "orange";',
+        importers: [
+          TestImporter((url) => Uri.parse("u:$url"), (url) {
+            var color = url.path;
+            return ImporterResult('.$color {color: $color}', indented: false);
+          })
+        ],
+        sourceMap: true);
 
     expect(
         result.sourceMap!.urls,