diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-15 02:31:53 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-15 02:31:53 -0500 |
| commit | 1e59e346ab7e15c07e3b176cefaa92dbadb53c27 (patch) | |
| tree | 45fed7ea813e135bb685515db91df8570d556776 /tests/test-external-open-lib-command.el | |
| parent | db522da5d27c9fe90cae64ab80c659068fbb29c2 (diff) | |
| download | dotemacs-1e59e346ab7e15c07e3b176cefaa92dbadb53c27.tar.gz dotemacs-1e59e346ab7e15c07e3b176cefaa92dbadb53c27.zip | |
refactor(org-webclipper): use setq, not setopt, for pandoc sleep time
`org-web-tools-pandoc-sleep-time' is a plain float with no
custom-set handler that needs to fire. `setopt' adds the entire
customize-variable validation machinery -- which, lazily, depends on
wid-edit being loaded.
The handler's tests stub `require' so org-web-tools never really
loads, then mock `setopt' via `cl-letf' on the function cell. That
mock has no effect on byte-compiled code because `setopt' is a
macro: the production handler has already expanded to a call into
`setopt--set'. When `setopt--set' runs, it walks into the customize
machinery and hits an unbound `widget-field-keymap' (wid-edit not
loaded), and the test fails with a confusing wrong-type-argument.
`setq' has identical runtime effect for this variable and dodges
the customize machinery entirely. Tests now pass without contorted
mocking.
Diffstat (limited to 'tests/test-external-open-lib-command.el')
0 files changed, 0 insertions, 0 deletions