feat(dart_frig_cli): add new command to the cli (#641)

Author: renancaraujo

bricks/dart_frog_new/__brick__/{{filename}}

@@ -1,7 +1,6 @@
 import 'package:dart_frog/dart_frog.dart';
-
 {{#is_route}}
 {{> route.dart}}
 {{/is_route}}{{#is_middleware}}
 {{> middleware.dart}}
-{{/is_middleware}}
+{{/is_middleware}}

bricks/dart_frog_new/hooks/lib/src/route_to_path.dart

@@ -1,3 +1,5 @@
+import 'package:path/path.dart' as path;
+
 /// Convert a route to a file path.
 ///
 /// If [preferIndex] is true, the path will be converted to a directory path
@@ -14,19 +16,23 @@ String routeToPath(
   String route, {
   bool preferIndex = false,
   String preamble = 'routes',
+  path.Context? pathContext,
 }) {
+  final context = pathContext ?? path.context;
+  final separator = context.separator;
+
   if (route == '/') {
-    return '$preamble/index.dart';
+    return '$preamble${separator}index.dart';
   }
 
-  final path =
-      route.split('/').where((element) => element.isNotEmpty).join('/');
+  final p =
+      route.split('/').where((element) => element.isNotEmpty).join(separator);
 
   if (preferIndex) {
-    final pathWithIndex = '$path/index.dart';
-    return '$preamble/$pathWithIndex';
+    final pathWithIndex = context.join(p, 'index.dart');
+    return context.join(preamble, pathWithIndex);
   }
 
-  final pathWithExtension = '$path.dart';
-  return '$preamble/$pathWithExtension';
+  final pathWithExtension = '$p.dart';
+  return context.join(preamble, pathWithExtension);
 }

bricks/dart_frog_new/hooks/test/src/route_to_path_test.dart

@@ -4,6 +4,12 @@ import 'package:test/test.dart';
 
 void main() {
   group('routeToPath', () {
+    test('defaults to current platform path context', () {
+      expect(routeToPath('/'), equals(path.join('routes', 'index.dart')));
+    });
+  });
+
+  group('routeToPath posix', () {
     group('preferIndex false', () {
       final expectedRouteToPathMappings = <String, String>{
         '/': '../routes/index.dart',
@@ -44,4 +50,55 @@ void main() {
       }
     });
   });
+
+  group('routeToPath windows', () {
+    group('preferIndex false', () {
+      final expectedRouteToPathMappings = <String, String>{
+        '/': r'..\routes\index.dart',
+        '/hello': r'..\routes\hello.dart',
+        '/hello/world': r'..\routes\hello\world.dart',
+        '/hello/[name]': r'..\routes\hello\[name].dart',
+        '/[id]/item': r'..\routes\[id]\item.dart',
+        '/[id]/part/item': r'..\routes\[id]\part\item.dart',
+      };
+
+      for (final entry in expectedRouteToPathMappings.entries) {
+        test('maps ${entry.key} -> ${entry.value}', () {
+          expect(
+            routeToPath(
+              entry.key,
+              preamble: r'..\routes',
+              pathContext: path.windows,
+            ),
+            equals(path.windows.normalize(entry.value)),
+          );
+        });
+      }
+    });
+
+    group('preferIndex true', () {
+      final expectedRouteToPathMappings = <String, String>{
+        '/': r'..\routes\index.dart',
+        '/hello': r'..\routes\hello\index.dart',
+        '/hello/world': r'..\routes\hello\world\index.dart',
+        '/hello/[name]': r'..\routes\hello\[name]\index.dart',
+        '/[id]/item': r'..\routes\[id]\item\index.dart',
+        '/[id]/part/item': r'..\routes\[id]\part\item\index.dart',
+      };
+
+      for (final entry in expectedRouteToPathMappings.entries) {
+        test('maps ${entry.key} -> ${entry.value}', () {
+          expect(
+            routeToPath(
+              entry.key,
+              preferIndex: true,
+              preamble: r'..\routes',
+              pathContext: path.windows,
+            ),
+            equals(path.windows.normalize(entry.value)),
+          );
+        });
+      }
+    });
+  });
 }

packages/dart_frog_cli/e2e/analysis_options.yaml

@@ -0,0 +1 @@
+include: package:very_good_analysis/analysis_options.4.0.0.yaml

packages/dart_frog_cli/e2e/pubspec.yaml

@@ -3,7 +3,7 @@ description: End to End tests for package:dart_frog_cli
 publish_to: none
 
 environment:
-  sdk: ">=2.17.0 <3.0.0"
+  sdk: ">=2.19.0 <3.0.0"
 
 dev_dependencies:
   dart_frog:
@@ -12,4 +12,4 @@ dev_dependencies:
   meta: ^1.7.0
   path: ^1.8.2
   test: ^1.19.2
-  very_good_analysis: ^3.0.0
+  very_good_analysis: ^4.0.0

packages/dart_frog_cli/e2e/test/dart_frog_build_test.dart renamed to packages/dart_frog_cli/e2e/test/build_test.dart

@@ -0,0 +1,46 @@
+import 'dart:io';
+
+import 'package:test/test.dart';
+
+Future<void> dartFrogNewRoute(
+  String routePath, {
+  required Directory directory,
+}) =>
+    _dartFrogNew(
+      routePath: routePath,
+      what: 'route',
+      directory: directory,
+    );
+
+Future<void> dartFrogNewMiddleware(
+  String routePath, {
+  required Directory directory,
+}) =>
+    _dartFrogNew(
+      routePath: routePath,
+      what: 'middleware',
+      directory: directory,
+    );
+
+Future<void> _dartFrogNew({
+  required String routePath,
+  required String what,
+  required Directory directory,
+}) async {
+  final result = await Process.run(
+    'dart_frog',
+    ['new', what, routePath],
+    workingDirectory: directory.path,
+    runInShell: true,
+  );
+
+  if (result.exitCode != 0) {
+    throw FormatException('${result.stderr}');
+  }
+}
+
+Matcher failsWithA({required String message}) {
+  return throwsA(
+    isA<FormatException>().having((e) => e.message.trim(), 'message', message),
+  );
+}

packages/dart_frog_cli/e2e/test/dart_frog_create_test.dart renamed to packages/dart_frog_cli/e2e/test/create_test.dart

@@ -0,0 +1,19 @@
+import 'dart:io';
+import 'package:path/path.dart' as path;
+import 'package:test/test.dart';
+
+File fileAt(String filePath, {required Directory on}) {
+  return File(path.join(on.path, filePath));
+}
+
+final exists = FileExistsMatcher(isTrue);
+final doesNotExist = FileExistsMatcher(isFalse);
+
+class FileExistsMatcher extends CustomMatcher {
+  FileExistsMatcher(Matcher matcher)
+      : super(
+          'File exists',
+          'file exists',
+          isA<File>().having((p) => p.existsSync(), 'exists', matcher),
+        );
+}