Medzoner / medzoner-go / 24384157843

75%

Build Type

push

travis-ci

Committed by web-flow

Commit Message

Feat/refacto (#216)

* docs: add conventional commits skill for Claude

* test(mock): add mocks for Templater, Captcher and Validater interfaces

* test: add unit tests for handler, repository, config, entity, http_utils and database
Cover 8 previously untested packages:
- handler: IndexHandler (GET/POST, captcha, validation, render errors)
- handler: NotFoundHandler (404, render error)
- repository: MysqlContactRepository (save, begin/prepare/exec/commit errors)
- repository: TechnoJSONRepository (success, file not found, invalid JSON)
- config: NewConfig with env vars
- entity: Contact.Get()
- http_utils: ResponseError (500, 400, 404)
- database: DbMigration (driver errors, invalid action)
Add go-sqlmock dependency for MySQL repository tests.

* fix(handler): return 400 status on validation error in POST handler
The WriteHeader call was gated by view.FormMessage which is always
empty, so validation errors returned 200 instead of 400.

* test(godog): reactivate BDD integration tests
- Replace disabled flag with GODOG_INTEGRATION=1 env guard
- Rewrite featurecontext to use real HTTP client requests
- Start server in background goroutine on dedicated test port
- Wait for /healthz/live before running scenarios
- Fix feature expectations to match actual routes and behavior
- 6 scenarios, 20 steps all passing

* test(godog): add scenarios to improve integration test coverage
- Add body and header assertion steps to featurecontext
- Add GET with TOR-HOST header scenario
- Add POST with submit param scenario (skip processing path)
- Add POST validation error body check
- Add POST redirect location header check
- Add second 404 route scenario
- Add infra.feature: liveness, readiness, robots.txt, CSS static

* test: increase unit test coverage to 74.4%
- event: add empty UUID error case
- handler: add Prefix, captcha confirm false, POST with submit tests
- templater: add ExecuteTemplate failure case
- database: add nil driver instantiation case
- customtype: ... (continued)

Coverage Stats

176 of 285 new or added lines in 14 files covered. (61.75%)

7 existing lines in 2 files now uncovered.

501 of 771 relevant lines covered (64.98%)

4.15 hits per line