Fix suffix flag and prefix check (#86)

Resolves #83 and #84.

Author: ibraheemdev

src/tree.rs

@@ -199,7 +199,7 @@ impl<T> Node<T> {
             let next = remaining[0];
 
             // For parameters with a suffix, we have to find the matching suffix or create a new child node.
-            if matches!(state.node().node_type, NodeType::Param { .. }) {
+            if let NodeType::Param { suffix: has_suffix } = state.node().node_type {
                 let terminator = remaining
                     .iter()
                     .position(|&b| b == b'/')
@@ -248,8 +248,10 @@ impl<T> Node<T> {
                     priority: 1,
                     ..Node::default()
                 });
-                let has_suffix = !matches!(*suffix, b"" | b"/");
+
+                let has_suffix = has_suffix || !matches!(*suffix, b"" | b"/");
                 state.node_mut().node_type = NodeType::Param { suffix: has_suffix };
+
                 state = state.set_child(child);
 
                 // If this is the final route segment, insert the value.
@@ -409,6 +411,10 @@ impl<T> Node<T> {
     /// Returns `true` if there is a wildcard node that contains a prefix within the current route segment,
     /// i.e. before the next trailing slash
     fn prefix_wild_child_in_segment(&self) -> bool {
+        if matches!(self.node_type, NodeType::Root) && self.prefix.is_empty() {
+            return false;
+        }
+
         if self.prefix.ends_with(b"/") {
             self.children.iter().any(Node::prefix_wild_child_in_segment)
         } else {

tests/insert.rs

@@ -16,6 +16,12 @@ fn conflict(with: &'static str) -> InsertError {
     InsertError::Conflict { with: with.into() }
 }
 
+// https://github.com/ibraheemdev/matchit/issues/84
+#[test]
+fn root_prefix_issue() {
+    InsertTest(vec![("{foo}", Ok(())), ("{foo}suffix", Ok(()))]).run()
+}
+
 #[test]
 fn wildcard_conflict() {
     InsertTest(vec![

tests/match.rs

@@ -134,6 +134,16 @@ fn bare_catchall() {
     .run()
 }
 
+// https://github.com/ibraheemdev/matchit/issues/83
+#[test]
+fn param_suffix_flag_issue() {
+    MatchTest {
+        routes: vec!["/foo/{foo}suffix", "/foo/{foo}/bar"],
+        matches: vec![("/foo/barsuffix", "/foo/{foo}suffix", p! { "foo" => "bar" })],
+    }
+    .run()
+}
+
 #[test]
 fn normalized() {
     MatchTest {