umputun / cronn / 19360076957

75%

Build Type

push

github

Committed by web-flow

Commit Message

Add execution history with SQLite storage and web UI display (#51)

* feat: add execution history with SQLite storage and web UI display

- implement SQLite persistence for job execution records with configurable limits
- add web UI to display execution history with logs, exit codes, and timestamps
- separate notification and web log outputs using independent OutputCapture buffers
- rename ErrorWriter to OutputCapture to better reflect dual-purpose usage
- fix exit code extraction using errors.As to unwrap error chains correctly
- add comprehensive tests for output capture and persistence layers

* chore: set default exec-max-lines to 100, disable if web not enabled

* refactor: address code review feedback from gemini-code-assist

- rename SerError to SetError following Go naming conventions
- refactor multi-writer construction to eliminate variable shadowing
- make error handling consistent using inline declarations in migrate()
- fix error variable shadowing by using execErr and outErr names

* test: add comprehensive tests for handleExecutionLogs handler

- test valid request with execution logs
- test missing job id error path
- test invalid execution id error path
- test non-existent execution error path
- test execution ownership validation (403 forbidden)
- improves app/web coverage from 85.2% to 88.1% (+2.9%)
- handleExecutionLogs now has 100% coverage

* fix: include command output in resume failure notifications

- resumeInterrupted now captures notifyOutput from executeCommand
- combines error message with command output for notifications
- matches pattern used in runJobWithCommand for consistency
- adds comprehensive tests for resume notification behavior
- test verifies failed resume includes both error and output
- test verifies successful resume does not send notification

refactor: replace time.Sleep with require.Eventually in tests

- replace time.Sleep with require.Eventually for async assertions
- improves test reliability and redu... (continued)

Run Details

206 of 227 new or added lines in 5 files covered. (90.75%)

6 existing lines in 4 files now uncovered.

2482 of 3345 relevant lines covered (74.2%)

36.75 hits per line