Code review comment responses

Must have had a brain cramp before checking this stuff in. Had left changes in made during testing that
rrousselGit · Jul 24, 2023 · 272c42a · 272c42a
1 parent b5d4ddd
commit 272c42a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/packages/riverpod_graph/lib/src/analyze.dart b/packages/riverpod_graph/lib/src/analyze.dart
@@ -96,7 +96,8 @@ Future<void> analyze(
 bool verifyRootDirectoryExists(String rootDirectory) {
   if (!Directory(rootDirectory).existsSync()) {
     throw FileSystemException(
-        'Requested scanning target directory does not exist $rootDirectory');
+      'Requested scanning target directory does not exist $rootDirectory',
+    );
   }
   return true;
 }

diff --git a/packages/riverpod_graph/test/analyze_test.dart b/packages/riverpod_graph/test/analyze_test.dart
@@ -1,5 +1,3 @@
-// ignore_for_file: avoid_types_on_closure_parameters
-
 import 'dart:io';
 
 import 'package:riverpod_graph/src/analyze.dart';
@@ -9,9 +7,8 @@ import 'package:test/test.dart';
 void main() {
   group('flutter graph analyzer tests', () {
     test('throw exception if analysis target directory does not exist', () {
-      // expect(() => verifyRootDirectoryExists('dogfood'),
-      //     throwsA(isA<FileSystemException>())
-      //     );
+      expect(() => verifyRootDirectoryExists('dogfood'),
+          throwsA(isA<FileSystemException>()));
     });
 
     test('returns true if analysis target directory does exist', () {