umputun / cronn / 19323121139

74%

master: 75%

Build Type

Pull #51

github

Committed by umputun

Commit Message

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 reduces flakiness
- tests run faster (ManualTrigger: 1.6s -> 0.95s, reload: 0.2s -> 0.01s)
- affected tests: manual trigger, reload, resumeInterrupted
- keep time.Sleep only for intentional delays (e.g., timeout simulation)

related to external review feedback

Pull Request Pull Request #51: Add execution history with SQLite storage and web UI display

Run Details

222 of 244 new or added lines in 5 files covered. (90.98%)

6 existing lines in 4 files now uncovered.

2502 of 3370 relevant lines covered (74.24%)

35.44 hits per line