umputun / cronn / 19256451851

75%

Build Type

push

github

Committed by umputun

Commit Message

chore: fix testifylint and wrapcheck linter warnings

Address 45 linter warnings (33 testifylint + 12 wrapcheck):

testifylint fixes:
- use assert.Len instead of assert.Equal with len()
- use assert.Empty instead of assert.Equal with empty strings
- use assert.True/False instead of assert.Equal with booleans
- use assert.GreaterOrEqual/Less instead of assert.True with comparisons
- use assert.InDelta for float comparisons
- use require.Error/NoError for error assertions
- fix expected-actual parameter order
- use assert.Positive/NotEmpty where appropriate

wrapcheck fixes:
- wrap errors with fmt.Errorf in file operations (crontab, resumer)
- wrap errors in I/O operations (log_prefixer)
- wrap errors in command execution and notifications (service)
- wrap errors in authentication (auth)
- wrap errors in database operations (sqlite)
- wrap errors in web server (web)

Also update 2 tests to match new wrapped error messages.

Run Details

19 of 33 new or added lines in 7 files covered. (57.58%)

38 existing lines in 2 files now uncovered.

2345 of 3133 relevant lines covered (74.85%)

35.72 hits per line