story-kit: merge 107_story_test_coverage_http_assets_rs_to_85

server/src/http/assets.rs

@@ -77,10 +77,10 @@ mod tests {
         // In release builds (with embedded dist/) this returns 200.
         // In debug builds without a built frontend dist/ it returns 404.
         let response = serve_embedded("__nonexistent_spa_route__.html");
+        let status = response.status();
         assert!(
-            response.status() == StatusCode::OK || response.status() == StatusCode::NOT_FOUND,
-            "unexpected status: {}",
-            response.status()
+            status == StatusCode::OK || status == StatusCode::NOT_FOUND,
+            "unexpected status: {status}",
         );
     }
 
@@ -95,10 +95,10 @@ mod tests {
     fn serve_embedded_does_not_panic_on_empty_path() {
         // Empty path normalises to index.html; OK in release, 404 in debug without dist/
         let response = serve_embedded("");
+        let status = response.status();
         assert!(
-            response.status() == StatusCode::OK || response.status() == StatusCode::NOT_FOUND,
-            "unexpected status: {}",
-            response.status()
+            status == StatusCode::OK || status == StatusCode::NOT_FOUND,
+            "unexpected status: {status}",
         );
     }
 
@@ -110,4 +110,39 @@ mod tests {
         let _files: Vec<_> = EmbeddedAssets::iter().collect();
         // No assertion needed – the test passes as long as it compiles and does not panic.
     }
+
+    #[tokio::test]
+    async fn embedded_index_handler_returns_ok_or_not_found() {
+        // Route the handler through TestClient; index.html is the SPA entry point.
+        let app = poem::Route::new().at("/", poem::get(embedded_index));
+        let cli = poem::test::TestClient::new(app);
+        let resp = cli.get("/").send().await;
+        let status = resp.0.status();
+        assert!(
+            status == StatusCode::OK || status == StatusCode::NOT_FOUND,
+            "unexpected status: {status}",
+        );
+    }
+
+    #[tokio::test]
+    async fn embedded_file_handler_with_path_returns_ok_or_not_found() {
+        // Non-asset paths fall back to index.html (SPA routing) or 404.
+        let app = poem::Route::new().at("/*path", poem::get(embedded_file));
+        let cli = poem::test::TestClient::new(app);
+        let resp = cli.get("/__spa_route__").send().await;
+        let status = resp.0.status();
+        assert!(
+            status == StatusCode::OK || status == StatusCode::NOT_FOUND,
+            "unexpected status: {status}",
+        );
+    }
+
+    #[tokio::test]
+    async fn embedded_asset_handler_missing_file_returns_not_found() {
+        // The assets/ prefix disables SPA fallback; missing files must return 404.
+        let app = poem::Route::new().at("/assets/*path", poem::get(embedded_asset));
+        let cli = poem::test::TestClient::new(app);
+        let resp = cli.get("/assets/__nonexistent__.js").send().await;
+        assert_eq!(resp.0.status(), StatusCode::NOT_FOUND);
+    }
 }