Fix router

packages/yew-router/src/service.rs

@@ -35,8 +35,10 @@ where
 fn push_impl(url: String) {
     let history = yew::utils::window().history().expect("no history");
 
+    let path = build_path_with_base(&url);
+
     history
-        .push_state_with_url(&JsValue::NULL, "", Some(&build_path_with_base(&url)))
+        .push_state_with_url(&JsValue::NULL, "", Some(&path))
         .expect("push history");
     let event = Event::new("popstate").unwrap();
     yew::utils::window()

packages/yew-router/src/utils.rs

@@ -1,15 +1,5 @@
 use wasm_bindgen::JsCast;
 
-fn strip_slash(path: String) -> String {
-    if path != "/" {
-        path.strip_suffix("/")
-            .map(|it| it.to_string())
-            .unwrap_or(path)
-    } else {
-        path
-    }
-}
-
 pub fn base_url() -> Option<String> {
     match yew::utils::document().query_selector("base[href]") {
         Ok(Some(base)) => {
@@ -18,15 +8,27 @@ pub fn base_url() -> Option<String> {
             let url = web_sys::Url::new(&base).unwrap();
             let base = url.pathname();
 
-            let base = strip_slash(base);
+            let base = if base != "/" {
+                base.strip_suffix("/")
+                    .map(|it| it.to_string())
+                    .unwrap_or(base)
+            } else {
+                return None;
+            };
+
             Some(base)
         }
         _ => None,
     }
 }
 
 pub fn build_path_with_base(to: &str) -> String {
-    let to = format!("{}{}", base_url().as_deref().unwrap_or(""), to);
-
-    strip_slash(to)
+    format!(
+        "{}{}",
+        base_url()
+            .as_deref()
+            .map(|it| it.strip_suffix("/").unwrap_or(it))
+            .unwrap_or(""),
+        to
+    )
 }

packages/yew-router/tests/parser.rs

@@ -20,6 +20,8 @@ enum Routes {
 
 #[test]
 fn test_base_url() {
+    document().head().unwrap().set_inner_html(r#""#);
+
     assert_eq!(base_url(), None);
 
     document()
@@ -64,3 +66,24 @@ fn test_get_query_params() {
         map
     });
 }
+
+#[test]
+fn test_build_base_url() {
+    document().head().unwrap().set_inner_html(r#""#);
+
+    assert_eq!(build_path_with_base("/posts"), "/posts");
+
+    document()
+        .head()
+        .unwrap()
+        .set_inner_html(r#"<base href="/router">"#);
+
+    assert_eq!(build_path_with_base("/posts/"), "/router/posts/");
+
+    document()
+        .head()
+        .unwrap()
+        .set_inner_html(r#"<base href="/">"#);
+
+    assert_eq!(build_path_with_base("/posts"), "/posts");
+}

packages/yew-router/tests/router.rs

@@ -82,8 +82,7 @@ fn component() -> Html {
 // - 404 redirects
 #[test]
 fn router_works() {
-    let app = App::<Comp>::new();
-    app.mount(yew::utils::document().get_element_by_id("output").unwrap());
+    yew::start_app_in_element::<Comp>(yew::utils::document().get_element_by_id("output").unwrap());
 
     assert_eq!("Home", obtain_result_by_id("result"));