38 lines
2.1 KiB
Markdown
38 lines
2.1 KiB
Markdown
# QA Report: CrowdSec Hub Preset (feature/beta-release)
|
|
|
|
**Date:** December 8, 2025
|
|
**QA Agent:** QA_Security
|
|
**Scope:** Post-merge QA after CrowdSec hub preset backend/frontend changes on `feature/beta-release`.
|
|
**Requested Steps:** `pre-commit run --all-files`, `backend: go test ./...`, `frontend: npm run test:ci`.
|
|
|
|
## Executive Summary
|
|
|
|
**Final Verdict:** ✅ PASS (coverage gate met)
|
|
|
|
- `pre-commit run --all-files` passes; coverage hook reports 85.0% vs required 85% (gate met) after adding middleware sanitize tests. Hooks include Go vet, version check, frontend type-check, and lint fix.
|
|
- `go test ./...` (backend) passes via task `Go: Test Backend`.
|
|
- `npm run test:ci` passes (Vitest, 70 files / 598 tests). React Query undefined-data warnings and jsdom navigation warnings appear but suites stay green.
|
|
|
|
## Test Results
|
|
|
|
| Area | Status | Notes |
|
|
| --- | --- | --- |
|
|
| Pre-commit | ✅ PASS | Coverage gate satisfied at 85.0% (minimum 85%) after middleware sanitize tests; all hooks succeeded. |
|
|
| Backend Unit Tests | ✅ PASS | `cd backend && go test ./...` (task: Go: Test Backend). |
|
|
| Frontend Unit Tests | ✅ PASS* | `npm run test:ci` (Vitest, 70 files / 598 tests). Warnings: React Query "query data cannot be undefined" for `securityConfig`/`securityRulesets`/`feature-flags`; jsdom "navigation to another Document". |
|
|
|
|
## Evidence / Logs
|
|
|
|
- Coverage hook output: `Computed coverage: 85.0% (minimum required 85%)` followed by “Coverage requirement met.”
|
|
- Backend tests: task output shows `ok github.com/Wikid82/charon/backend/internal/...` with no failures.
|
|
- Frontend Vitest: full log at [test-results/frontend-test.log](test-results/frontend-test.log) (70 files, 598 tests, warnings noted above).
|
|
|
|
## Follow-ups / Recommendations
|
|
|
|
1. Optionally tighten React Query mocks in Security and Layout suites to eliminate "query data cannot be undefined" warnings; consider default fixtures for `securityConfig`, `securityRulesets`, and `feature-flags`.
|
|
2. Silence jsdom "navigation to another Document" warnings if noise persists (e.g., stub navigation or avoid window.location changes in tests).
|
|
|
|
---
|
|
|
|
**Status:** ✅ QA Passed (coverage gate satisfied).
|