diff options
| author | Craig Jennings <c@cjennings.net> | 2025-11-08 13:16:42 -0600 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2025-11-08 13:20:18 -0600 |
| commit | b6761ed66012fcf3e98aa252582aaf03b2b85c12 (patch) | |
| tree | 0e7c55e734e04de8404ea89ebfa08ed3740a492b /tests/test-wttrin--add-buffer-instructions.el | |
| parent | 62bde926991ffc2fb57b49c0618b2dd177765dd6 (diff) | |
refactor: Improve cache cleanup clarity and add input validation
Cache Refactorings:
- Extract wttrin--get-cache-entries-by-age helper function
- Refactor wttrin--cleanup-cache-if-needed to use helper
- Improves code clarity with descriptive variable names
- Makes cache logic independently testable
- Add 15 new tests (8 for helper, 7 for refactored cleanup)
Input Validation:
- Add range validation to wttrin-mode-line-startup-delay (1-10 seconds)
- Prevents invalid values (0, negative, or excessive delays)
- Uses restricted-sexp type for enforcement
Test Fixes:
- Fix debug function naming (use public wttrin-debug-clear-log)
- Ensure wttrin-debug module loads in test setup
- Make mocks synchronous for batch mode compatibility
- Update test expectations to match actual debug messages
- Fix startup delay test to check defcustom default value
Linter/Compiler Fixes:
- Add declare-function for wttrin--debug-log
- Add defvar/declare-function in wttrin-debug.el
- Fix docstring line lengths (wrap at 80 chars)
- Fix checkdoc warnings (imperative mood, escaping)
- Disambiguate temporary-file-directory reference
All 165 tests passing. Clean byte-compile, package-lint, checkdoc.
Diffstat (limited to 'tests/test-wttrin--add-buffer-instructions.el')
0 files changed, 0 insertions, 0 deletions
