Add automated route and frontend coverage

Author: blackopsrepl

.forgejo/workflows/ci.yml

@@ -33,7 +33,9 @@ jobs:
         run: cargo build
 
       - name: Run tests
-        run: cargo test
+        run: |
+          cargo test
+          node --test tests/*.test.js
 
       - name: Verify package contents
         run: ./scripts/verify-package.sh

.forgejo/workflows/release.yml

@@ -29,7 +29,9 @@ jobs:
         run: cargo build --release
 
       - name: Run tests
-        run: cargo test
+        run: |
+          cargo test
+          node --test tests/*.test.js
 
   create-release:
     name: Create Release

.github/workflows/ci.yml

@@ -47,7 +47,9 @@ jobs:
         run: cargo build
 
       - name: Run tests
-        run: cargo test
+        run: |
+          cargo test
+          node --test tests/*.test.js
 
       - name: Verify package contents
         run: ./scripts/verify-package.sh

.github/workflows/release.yml

@@ -48,7 +48,9 @@ jobs:
         run: cargo build --release
 
       - name: Run tests
-        run: cargo test
+        run: |
+          cargo test
+          node --test tests/*.test.js
 
   create-release:
     name: Create GitHub Release

Cargo.toml

@@ -24,4 +24,4 @@ include_dir = "0.7"
 
 [dev-dependencies]
 tokio = { version = "1", features = ["macros", "rt-multi-thread"] }
-tower = "0.5"
+tower = { version = "0.5", features = ["util"] }

Makefile

@@ -27,7 +27,7 @@ VERSIONED_CSS := static/sf/sf.$(VERSION).css
 VERSIONED_JS := static/sf/sf.$(VERSION).js
 
 # ============== Phony Targets ==============
-.PHONY: banner help assets build build-release test test-quick test-doc test-unit test-one \
+.PHONY: banner help assets build build-release test test-quick test-doc test-unit test-frontend test-one \
         lint fmt fmt-check clippy ci-local pre-release version package-verify \
         bump-patch bump-minor bump-major bump-dry demo-serve \
         publish-dry publish clean watch
@@ -84,6 +84,7 @@ test: banner
 	@printf "$(CYAN)$(BOLD)╚══════════════════════════════════════╝$(RESET)\n\n"
 	@printf "$(ARROW) $(BOLD)Running all tests...$(RESET)\n"
 	@cargo test && \
+		node --test tests/*.test.js && \
 		printf "\n$(GREEN)$(CHECK) All tests passed$(RESET)\n\n" || \
 		(printf "\n$(RED)$(CROSS) Tests failed$(RESET)\n\n" && exit 1)
 
@@ -99,6 +100,10 @@ test-quick: banner
 	@cargo test --lib --quiet && \
 		printf "$(GREEN)$(CHECK) Unit tests passed$(RESET)\n\n" || \
 		(printf "$(RED)$(CROSS) Unit tests failed$(RESET)\n\n" && exit 1)
+	@printf "$(PROGRESS) Running frontend tests...\n"
+	@node --test tests/*.test.js && \
+		printf "$(GREEN)$(CHECK) Frontend tests passed$(RESET)\n\n" || \
+		(printf "$(RED)$(CROSS) Frontend tests failed$(RESET)\n\n" && exit 1)
 
 test-doc:
 	@printf "$(PROGRESS) Running doctests...\n"
@@ -112,6 +117,12 @@ test-unit:
 		printf "$(GREEN)$(CHECK) Unit tests passed$(RESET)\n" || \
 		(printf "$(RED)$(CROSS) Unit tests failed$(RESET)\n" && exit 1)
 
+test-frontend:
+	@printf "$(PROGRESS) Running frontend tests...\n"
+	@node --test tests/*.test.js && \
+		printf "$(GREEN)$(CHECK) Frontend tests passed$(RESET)\n" || \
+		(printf "$(RED)$(CROSS) Frontend tests failed$(RESET)\n" && exit 1)
+
 test-one:
 	@printf "$(PROGRESS) Running test: $(YELLOW)$(TEST)$(RESET)\n"
 	@RUST_LOG=info cargo test $(TEST) -- --nocapture
@@ -155,8 +166,10 @@ ci-local: banner
 	@$(MAKE) clippy --no-print-directory
 	@printf "$(PROGRESS) Step 5/6: Doctests...\n"
 	@cargo test --doc --quiet && printf "$(GREEN)$(CHECK) Doctests passed$(RESET)\n"
-	@printf "$(PROGRESS) Step 6/6: Unit tests...\n"
+	@printf "$(PROGRESS) Step 6/7: Unit tests...\n"
 	@cargo test --lib --quiet && printf "$(GREEN)$(CHECK) Unit tests passed$(RESET)\n"
+	@printf "$(PROGRESS) Step 7/7: Frontend tests...\n"
+	@node --test tests/*.test.js && printf "$(GREEN)$(CHECK) Frontend tests passed$(RESET)\n"
 	@printf "\n$(GREEN)$(BOLD)╔══════════════════════════════════════════════════════════╗$(RESET)\n"
 	@printf "$(GREEN)$(BOLD)║              $(CHECK) CI SIMULATION PASSED                      ║$(RESET)\n"
 	@printf "$(GREEN)$(BOLD)╚══════════════════════════════════════════════════════════╝$(RESET)\n\n"
@@ -196,7 +209,7 @@ pre-release: banner
 	@$(MAKE) fmt-check --no-print-directory
 	@$(MAKE) clippy --no-print-directory
 	@printf "$(PROGRESS) Running full test suite...\n"
-	@cargo test --quiet && printf "$(GREEN)$(CHECK) All tests passed$(RESET)\n"
+	@cargo test --quiet && node --test tests/*.test.js && printf "$(GREEN)$(CHECK) All tests passed$(RESET)\n"
 	@printf "$(PROGRESS) Dry-run publish...\n"
 	@cargo publish --dry-run 2>&1 | tail -1
 	@printf "$(PROGRESS) Verifying packaged contents...\n"
@@ -273,6 +286,7 @@ help: banner
 	@/bin/echo -e "  $(GREEN)make test-quick$(RESET)     - Run doctests + unit tests (fast)"
 	@/bin/echo -e "  $(GREEN)make test-doc$(RESET)       - Run doctests only"
 	@/bin/echo -e "  $(GREEN)make test-unit$(RESET)      - Run unit tests only"
+	@/bin/echo -e "  $(GREEN)make test-frontend$(RESET)  - Run frontend Node tests"
 	@/bin/echo -e "  $(GREEN)make test-one TEST=name$(RESET) - Run specific test with output"
 	@/bin/echo -e ""
 	@/bin/echo -e "$(CYAN)$(BOLD)Lint & Format:$(RESET)"

README.md

@@ -53,6 +53,14 @@ Every backend element has a corresponding UI element. The library grows
 alongside the solver. When you scaffold a new SolverForge project with
 `solverforge new`, it's already wired in.
 
+## Testing
+
+Repository coverage now includes the embedded Rust asset routes plus Node-based
+frontend tests for backend adapters, solver lifecycle transitions, and core
+component rendering. Use `make test` for the full suite, `make test-quick` for
+Rust doctests and unit tests plus frontend coverage, or `make test-frontend`
+when you only want the JavaScript suite.
+
 ## Quick Start
 
 ```html

src/lib.rs

@@ -192,4 +192,93 @@ mod tests {
 
         assert_eq!(missing_resp.status(), StatusCode::NOT_FOUND);
     }
+
+    #[tokio::test]
+    async fn serves_top_level_assets_with_short_cache_and_expected_mime() {
+        let response = routes()
+            .oneshot(
+                Request::builder()
+                    .uri("/sf/sf.css")
+                    .body(Body::empty())
+                    .unwrap(),
+            )
+            .await
+            .unwrap();
+
+        assert_eq!(response.status(), StatusCode::OK);
+        assert_eq!(
+            response.headers().get(header::CONTENT_TYPE).unwrap(),
+            "text/css; charset=utf-8"
+        );
+        assert_eq!(
+            response.headers().get(header::CACHE_CONTROL).unwrap(),
+            "public, max-age=3600"
+        );
+
+        let body = to_bytes(response.into_body(), usize::MAX).await.unwrap();
+        let css = String::from_utf8(body.to_vec()).unwrap();
+        assert!(css.contains("--sf-emerald-50"));
+        assert!(css.contains(".sf-gantt-split"));
+    }
+
+    #[tokio::test]
+    async fn serves_immutable_assets_with_long_cache_and_expected_mime() {
+        let image = routes()
+            .oneshot(
+                Request::builder()
+                    .method(Method::GET)
+                    .uri("/sf/img/ouroboros.svg")
+                    .body(Body::empty())
+                    .unwrap(),
+            )
+            .await
+            .unwrap();
+
+        assert_eq!(image.status(), StatusCode::OK);
+        assert_eq!(
+            image.headers().get(header::CONTENT_TYPE).unwrap(),
+            "image/svg+xml"
+        );
+        assert_eq!(
+            image.headers().get(header::CACHE_CONTROL).unwrap(),
+            "public, max-age=31536000, immutable"
+        );
+
+        let vendor = routes()
+            .oneshot(
+                Request::builder()
+                    .method(Method::GET)
+                    .uri("/sf/vendor/frappe-gantt/frappe-gantt.min.js")
+                    .body(Body::empty())
+                    .unwrap(),
+            )
+            .await
+            .unwrap();
+
+        assert_eq!(vendor.status(), StatusCode::OK);
+        assert_eq!(
+            vendor.headers().get(header::CONTENT_TYPE).unwrap(),
+            "application/javascript; charset=utf-8"
+        );
+        assert_eq!(
+            vendor.headers().get(header::CACHE_CONTROL).unwrap(),
+            "public, max-age=31536000, immutable"
+        );
+    }
+
+    #[tokio::test]
+    async fn returns_not_found_for_missing_assets() {
+        let response = routes()
+            .oneshot(
+                Request::builder()
+                    .method(Method::GET)
+                    .uri("/sf/does-not-exist.js")
+                    .body(Body::empty())
+                    .unwrap(),
+            )
+            .await
+            .unwrap();
+
+        assert_eq!(response.status(), StatusCode::NOT_FOUND);
+    }
 }