feat(mason_cli): improve error when running add in an uninitialized w…

…orkspace (#677)
felangel · Dec 23, 2022 · 05baddb · 05baddb
1 parent a355896
commit 05baddb
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/packages/mason_cli/lib/src/install_brick.dart b/packages/mason_cli/lib/src/install_brick.dart
@@ -9,7 +9,11 @@ mixin InstallBrickMixin on MasonCommand {
   /// Installs a specific brick and returns the [CachedBrick] reference.
   Future<CachedBrick> addBrick(Brick brick, {bool global = false}) async {
     final bricksJson = global ? globalBricksJson : localBricksJson;
-    if (bricksJson == null) usageException('bricks.json not found');
+    if (bricksJson == null) {
+      usageException(
+        'Mason has not been initialized.\nDid you forget to run mason init?',
+      );
+    }
 
     final masonLockJsonFile =
         global ? globalMasonLockJsonFile : localMasonLockJsonFile;

diff --git a/packages/mason_cli/test/commands/add_test.dart b/packages/mason_cli/test/commands/add_test.dart
@@ -52,7 +52,11 @@ void main() {
       Directory.current = Directory.systemTemp.createTempSync();
       final result = await commandRunner.run(['add', 'example', '--path', '.']);
       expect(result, equals(ExitCode.usage.code));
-      verify(() => logger.err('bricks.json not found')).called(1);
+      verify(
+        () => logger.err(
+          'Mason has not been initialized.\nDid you forget to run mason init?',
+        ),
+      ).called(1);
     });
 
     test('exits with code 64 when exception occurs', () async {