fix(mason): allow optional __brick__ directory (#616)

felangel · Nov 12, 2022 · 4e1ac92 · 4e1ac92
1 parent 98a958a
commit 4e1ac92
Show file tree

Hide file tree

Showing 3 changed files with 35 additions and 20 deletions.
diff --git a/packages/mason/lib/src/generator.dart b/packages/mason/lib/src/generator.dart
@@ -84,26 +84,28 @@ class MasonGenerator extends Generator {
       file.readAsStringSync(),
       (m) => BrickYaml.fromJson(m!),
     ).copyWith(path: file.path);
-    final brickDirectory = p.join(path, BrickYaml.dir);
-    final brickFiles = Directory(brickDirectory)
-        .listSync(recursive: true)
-        .whereType<File>()
-        .map((file) {
-      return () async {
-        final resource = await _descriptorPool.request();
-        try {
-          final content = await File(file.path).readAsBytes();
-          final relativePath = file.path.substring(
-            file.path.indexOf(BrickYaml.dir) + 1 + BrickYaml.dir.length,
-          );
-          return TemplateFile.fromBytes(relativePath, content);
-        } on Exception {
-          return null;
-        } finally {
-          resource.release();
-        }
-      }();
-    });
+    final brickDirectory = Directory(p.join(path, BrickYaml.dir));
+    final brickFiles = brickDirectory.existsSync()
+        ? brickDirectory
+            .listSync(recursive: true)
+            .whereType<File>()
+            .map((file) {
+            return () async {
+              final resource = await _descriptorPool.request();
+              try {
+                final content = await File(file.path).readAsBytes();
+                final relativePath = file.path.substring(
+                  file.path.indexOf(BrickYaml.dir) + 1 + BrickYaml.dir.length,
+                );
+                return TemplateFile.fromBytes(relativePath, content);
+              } on Exception {
+                return null;
+              } finally {
+                resource.release();
+              }
+            }();
+          })
+        : <Future<TemplateFile?>>[];
 
     return MasonGenerator(
       brickYaml.name,

diff --git a/packages/mason/test/fixtures/empty/brick.yaml b/packages/mason/test/fixtures/empty/brick.yaml
@@ -0,0 +1,3 @@
+name: empty
+description: An empty brick
+version: 0.1.0+1
diff --git a/packages/mason/test/src/generator_test.dart b/packages/mason/test/src/generator_test.dart
@@ -36,6 +36,16 @@ void main() {
         expect(files, isEmpty);
       });
 
+      test('constructs an instance (empty)', () async {
+        final brick = Brick.path(path.join('test', 'fixtures', 'empty'));
+        final generator = await MasonGenerator.fromBrick(brick);
+        final tempDir = Directory.systemTemp.createTempSync();
+        final files = await generator.generate(
+          DirectoryGeneratorTarget(tempDir),
+        );
+        expect(files, isEmpty);
+      });
+
       test('constructs an instance (hello_world)', () async {
         const name = 'Dash';
         final brick = Brick.path(