Sergey Vanyushkin
7ff3fa0992
All checks were successful
ci/woodpecker/pr/pipeline Pipeline was successful
Implement full comments system: domain entities (Comment, CommentLike), value objects (CommentContent), use cases (CRUD, like toggle), SQLAlchemy repository, API v1 endpoints, web UI with comment form and nested replies, i18n translations (EN/RU/FR/DE), and E2E tests. Fix nested reply (reply-to-reply) not displaying — the flat reply_comments dict was only queried for top-level comment IDs, so deeply nested replies were saved to DB (incrementing comment count) but never rendered. Switch to a recursive Jinja2 macro that renders any nesting depth.
3.1 KiB
3.1 KiB
Test Model: Blog
Global test coverage map for the blog application. Use this file to assess which features are covered, where gaps exist, and what to prioritize when adding new tests.
Coverage Matrix
| Feature | Unit | Integration | API | E2E | Priority | Status |
|---|---|---|---|---|---|---|
| Post Lifecycle (CRUD, Publish) | 85% | — | 90% | 70% | P0 | ✅ Active |
| RBAC & Access Control | 100% | — | 90% | 60% | P0 | ✅ Active |
| Domain Value Objects | 100% | — | — | — | P0 | ✅ Stable |
| Domain Entities | 95% | — | — | — | P0 | ✅ Stable |
| Domain Exceptions | 100% | — | — | — | P1 | ✅ Stable |
| List & Search Posts | 70% | — | — | — | P1 | ⚠️ Partial |
| Keycloak Auth Client | 80% | — | — | — | P0 | ✅ Active |
| App Bootstrap & Config | 75% | — | — | — | P1 | ✅ Stable |
| Transaction Manager | 60% | — | — | — | P2 | ⚠️ Partial |
| Web UI Error Handling | — | — | — | 50% | P1 | ⚠️ Partial |
| Pagination | 40% | — | — | 60% | P1 | ⚠️ Partial |
| Post Edit via Web | — | — | — | 40% | P1 | ⚠️ Partial |
| Post Delete via Web | — | — | — | 40% | P1 | ⚠️ Partial |
| i18n Localization | 100% | — | — | — | P1 | ✅ Active |
| Post Likes | 100% | — | 100% | — | P1 | ✅ Active |
| Comments (CRUD, Like, Nested) | — | — | — | — | P1 | 🔴 In Progress |
Legend: ✅ Covered / ⚠️ Partial / ❌ Missing / — Not Applicable
Feature Files
| Feature | Model File |
|---|---|
| Post Lifecycle | FEATURE_POST_LIFECYCLE.md |
| RBAC & Access Control | FEATURE_RBAC.md |
| Domain Foundation | FEATURE_DOMAIN_FOUNDATION.md |
| Infrastructure & Bootstrap | FEATURE_INFRASTRUCTURE.md |
| i18n Localization | FEATURE_INFRASTRUCTURE.md |
| Post Likes | FEATURE_LIKES.md |
| Comments | FEATURE_COMMENTS.md |
Test Naming Convention
- TC-UNIT-NNN: Unit test case
- TC-E2E-NNN: End-to-end test case
- TC-INT-NNN: Integration test case
- TC-API-NNN: API test case
How to Update This Model
- When adding a new test, assign the next available TC-ID in the relevant feature file.
- Update the Coverage Matrix above if the new test closes a gap or changes coverage percentage.
- Update the
Last Verifiedfield in the feature file after running the test successfully. - When a test is deleted or renamed, update the corresponding TC entry and mark it as Deprecated.
Risk Areas
- No Integration Tests: SQLAlchemy repository has no integration tests against a real database.
- Restored API Tests: API endpoint tests restored in
tests/api/covering all CRUD, publish/unpublish, and RBAC policies. - Web UI Error Handling: Only covered indirectly via E2E; no dedicated error-scenario E2E tests.
- Pagination Edge Cases: Page boundaries, empty pages, and large offsets are not explicitly tested.
- Edit/Delete Web Flows: No E2E coverage for editing or deleting posts through the web UI.