[go_router] #127016 preserved route name case when caching _nameToPath

packages/go_router/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 8.0.3
+
+- Makes namedLocation and route name related APIs case sensitive.
+
 ## 8.0.2 
 
 - Fixes a bug in `debugLogDiagnostics` to support StatefulShellRoute.

packages/go_router/lib/src/configuration.dart

@@ -221,7 +221,7 @@ class RouteConfiguration {
           '${queryParameters.isEmpty ? '' : ', queryParameters: $queryParameters'}');
       return true;
     }());
-    final String keyName = name.toLowerCase();
+    final String keyName = name;
     assert(_nameToPath.containsKey(keyName), 'unknown route name: $name');
     final String path = _nameToPath[keyName]!;
     assert(() {
@@ -564,7 +564,7 @@ class RouteConfiguration {
         final String fullPath = concatenatePaths(parentFullPath, route.path);
 
         if (route.name != null) {
-          final String name = route.name!.toLowerCase();
+          final String name = route.name!;
           assert(
               !_nameToPath.containsKey(name),
               'duplication fullpaths for name '

packages/go_router/pubspec.yaml

@@ -1,7 +1,7 @@
 name: go_router
 description: A declarative router for Flutter based on Navigation 2 supporting
   deep linking, data-driven routes and more
-version: 8.0.2
+version: 8.0.3
 repository: https://github.com/flutter/packages/tree/main/packages/go_router
 issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+go_router%22
 

packages/go_router/test/name_case_test.dart

@@ -0,0 +1,66 @@
+// Copyright 2013 The Flutter Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+import 'package:flutter/material.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:go_router/go_router.dart';
+
+void main() {
+  testWidgets(
+    'Route names are case sensitive',
+    (WidgetTester tester) async {
+      // config router with 2 routes with the same name but different case (Name, name)
+      final GlobalKey<NavigatorState> navKey = GlobalKey<NavigatorState>();
+      final GoRouter router = GoRouter(
+        navigatorKey: navKey,
+        routes: <GoRoute>[
+          GoRoute(
+            path: '/',
+            name: 'Name',
+            builder: (_, __) => const ScreenA(),
+          ),
+          GoRoute(
+            path: '/path',
+            name: 'name',
+            builder: (_, __) => const ScreenB(),
+          ),
+        ],
+      );
+
+      // run MaterialApp, initial screen path is '/' -> ScreenA
+      await tester.pumpWidget(
+        MaterialApp.router(
+          routerConfig: router,
+          title: 'GoRouter Testcase',
+        ),
+      );
+
+      // go to ScreenB
+      navKey.currentContext!.goNamed('name');
+      assert(GoRouter.of(navKey.currentContext!).location == '/path');
+
+      // go to ScreenA
+      navKey.currentContext!.goNamed('Name');
+      assert(GoRouter.of(navKey.currentContext!).location == '/');
+    },
+  );
+}
+
+class ScreenA extends StatelessWidget {
+  const ScreenA({super.key});
+
+  @override
+  Widget build(BuildContext context) {
+    return Container();
+  }
+}
+
+class ScreenB extends StatelessWidget {
+  const ScreenB({super.key});
+
+  @override
+  Widget build(BuildContext context) {
+    return Container();
+  }
+}