huh

Author: adhami3310

tests/units/test_app.py

@@ -298,16 +298,11 @@ def test_add_page_invalid_api_route(app: App, index_page):
         app: The app to test.
         index_page: The index page.
     """
-    with pytest.raises(ValueError):
-        app.add_page(index_page, route="api")
-    with pytest.raises(ValueError):
-        app.add_page(index_page, route="/api")
-    with pytest.raises(ValueError):
-        app.add_page(index_page, route="/api/")
-    with pytest.raises(ValueError):
-        app.add_page(index_page, route="api/foo")
-    with pytest.raises(ValueError):
-        app.add_page(index_page, route="/api/foo")
+    app.add_page(index_page, route="api")
+    app.add_page(index_page, route="/api")
+    app.add_page(index_page, route="/api/")
+    app.add_page(index_page, route="api/foo")
+    app.add_page(index_page, route="/api/foo")
     # These should be fine
     app.add_page(index_page, route="api2")
     app.add_page(index_page, route="/foo/api")

tests/units/test_route.py

@@ -50,8 +50,6 @@ def test_catchall_in_route(route_name, expected):
     "route_name",
     [
         "/products",
-        "/products/[category]/[...]/details/[version]",
-        "[...]",
         "/products/details",
     ],
 )
@@ -62,6 +60,8 @@ def test_verify_valid_routes(route_name):
 @pytest.mark.parametrize(
     "route_name",
     [
+        "/products/[category]/[...]/details/[version]",
+        "[...]",
         "/products/[...]/details/[category]/latest",
         "/blog/[...]/post/[year]/latest",
         "/products/[...]/details/[...]/[category]/[...]/latest",