bugfix: Catch route not found in spanner (#137)

* catch route not found in spanner

* _

* bump package versions

Author: codekeyz

packages/pharaoh/pubspec.yaml

@@ -1,14 +1,14 @@
 name: pharaoh
 description: Minimalist web-server library for Dart
-version: 0.0.8+2
+version: 0.0.8+3
 repository: https://github.com/codekeyz/pharaoh/tree/main/packages/pharaoh
 
 environment:
   sdk: ^3.0.0
 
 dependencies:
   meta: ^1.15.0
-  spanner: ^1.0.4
+  spanner: ^1.0.5
   mime: ^1.0.4
   collection: ^1.18.0
   http_parser: ^4.0.2

packages/pharaoh/test/issue_route_not_found_test.dart

@@ -0,0 +1,17 @@
+import 'package:pharaoh/pharaoh.dart';
+import 'package:spookie/spookie.dart';
+
+void main() {
+  test('should error on route not found', () async {
+    final app = Pharaoh()..get('/', (req, res) => res.ok('Hello'));
+
+    final tester = await request(app);
+
+    await tester.get('/').expectStatus(200).expectBody('Hello').test();
+
+    await tester
+        .get('/come')
+        .expectStatus(404)
+        .expectJsonBody({"error": "Route not found: /come"}).test();
+  });
+}

packages/spanner/lib/src/tree/tree.dart

@@ -202,6 +202,10 @@ class Spanner {
 
       wildcardNode = childNode?.wildcardNode ?? wildcardNode;
 
+      if (childNode == null && parametricNode == null && wildcardNode == null) {
+        return RouteResult(resolvedParams, getResults(null));
+      }
+
       // set root node as current child
       rootNode = childNode ?? parametricNode ?? rootNode;
 

packages/spanner/pubspec.yaml

@@ -1,6 +1,6 @@
 name: spanner
 description: Generic HTTP Router implementation, internally uses a Radix Tree (aka compact Prefix Tree), supports route params, wildcards.
-version: 1.0.4
+version: 1.0.5
 repository: https://github.com/Pharaoh-Framework/pharaoh/tree/main/packages/spanner
 
 environment: