aboutsummaryrefslogtreecommitdiff
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* feat(agenda): add the renderer output profile and its cache fileCraig Jennings9 hours1-0/+235
| | | | | | | | | | | | | | | | | | The surface that reads this went live, and it reads a different shape than the canonical profile emits. - cj/agenda-render-json adds s and e in epoch milliseconds, plus t for the title. The canonical fields ride along, so one file serves both readers. - Every row gets an end it can be drawn with. An all-day entry spans its day. A point event has zero width. The canonical profile still reports null there, which is faithful to the source but not a width. - cj/agenda-render-cache-update writes today to the XDG cache path. Input stays epoch seconds on both profiles. Only the output converts, and the guards that catch a milliseconds-for-seconds caller still apply. Nothing schedules the cache write yet, so the file is only as fresh as its last call. Picking a refresh mechanism is its own decision.
* feat(agenda): query the agenda window as JSON for external renderersCraig Jennings9 hours3-0/+792
| | | | | | | | | | | | | | | | | | | | I need the agenda drawable from outside Emacs, on a 24-hour axis that refreshes every few minutes. - cj/agenda-window-json returns every entry intersecting an epoch-second window. - Epoch seconds keep timezone out of the interface. The consumer converts per city, so DST weeks stop being a special case. - Repeating entries expand to one row per occurrence through org's own arithmetic, so they land where my agenda shows them. - Given an out-path it writes atomically. A reader on a timer never sees a half-written file. - It skips archived and commented subtrees, as the agenda does. The trap: SCHEDULED and DEADLINE are properties on the planning element, not children in the parse tree. Mapping over 'timestamp returns neither, and does it silently. An agenda is mostly made of those two. The tests pin it with a fixture carrying no body timestamp.
* fix(agenda): make the project-name category actually applyCraig Jennings34 hours1-8/+103
| | | | | | | | | | | | | | | | | | | - The override never fired once since it shipped. - Its guard wanted org-category to already equal "todo". - Org leaves that nil and derives the name later. - So the condition was false in every real buffer. The guard now tests for nil, and the hook runs at depth -100. That depth is load-bearing. org-get-category resolves a deferred value the element cache then holds, so any hook reading it first freezes "todo" and leaves the override inert. add-hook prepends, and three hooks already run ahead. The old tests set org-category by hand to a value org never produces, so they passed while the feature did nothing. They now visit real files and assert on what the agenda reads. The two todo.org agenda files show their project name. Files with distinctive names keep theirs, and an explicit category still wins.
* fix(test): anchor monthly tests on a day every month hasCraig Jennings34 hours2-10/+33
| | | | | | | | | | | | | | | | | | - The tests anchored on tomorrow, whose day varies by run date. - A series on the 31st correctly skips months without one. - That gave seven occurrences against an assertion of ten. - The suite went red on the 30th and aborted the run. A relative offset is not a date-independent anchor, which is what the file header claimed it was. I added a helper that walks forward to the first day at or below the 28th, and pointed the three cadence tests at it. One assertion was also too weak to notice. It checked that each occurrence fell on or before the anchor day, which a skipped month satisfies without ever landing on the right day. It now checks equality. Verified across 400 simulated run dates: no anchor past the 28th and no count outside the expected band. The old anchor fails nine of those days.
* test(auto-dim): pin the minibuffer dim behavior, not just the settingCraig Jennings3 days1-5/+94
| | | | | | | | | | | | | | | - The old assertion checked a value the module just set. - It couldn't fail even if the fork inverted the flag. - Three behavioral tests now drive real windows and minibuffer. - One pins the working half, one the flag-is-t boundary. - The third pins a known fork gap as an expected failure. adob--rescan-windows ignores the flag, so a completion popup dims every window. I handed that fix to the fork, which owns the code. When the fork honors the flag, the pinned test reports an unexpected pass and can go. Each test asserts the other window is still dimmed. Without that control, three broken implementations look exactly like a working fix.
* fix(telega): alert on server death and pin the container imageCraig Jennings4 days2-0/+224
| | | | | | | | | | | | | | - A dead telega-server now raises a persistent desktop notification. - The container image is pinned by digest, not by tag. - A clean telega exit stays silent, so quitting never pages. telega reports an abnormal server exit through the echo area, where it scrolls away unseen. That's how a dead server reads as a quiet Telegram. The scan stops partway, and the chats it never reached look empty. It's happened twice. Both times something downstream caught it rather than the failure announcing itself. telega pins its image only when its TDLib min and max versions match. Here the max is nil, so the image was always the latest tag. A floating tag against a fixed elpa package means the server can change underneath it with nothing announcing that either. Neither change fixes the crash. The root cause is still open. The assertion is a plist parse failure. Its coredump shows a smashed stack, which reads more like memory corruption than a parse rejection. I didn't pull a newer image, which was the other half of the plan. The pull was meant to test whether an old server was failing against newer elisp. The frozen image ships libtdjson 1.8.64, exactly the version this telega requires. That theory no longer holds, so pulling would trade a matched server for an unknown one.
* refactor(agenda): replace the dedicated frame with a full-frame F8Craig Jennings4 days4-1280/+215
| | | | | | | | | | | | - F8 now fills the whole frame instead of three quarters. - The agenda rebuilds itself on every five-minute wall-clock mark. - Quitting restores the window layout it took over. - The dedicated agenda frame and its tests are gone. - S-<f8> returns to the force-rescan. The frame bought live shared state. It paid for that with a read-only deny policy, engage-routing and a snapshot failure path. All of it existed only because the agenda shared a process with my working frames. A full-frame F8 needs none of it. The refresh rebuilds only a visible agenda. It keeps point on its line. The body is guarded, because a signal in a repeating timer resignals every tick.
* fix: restore Full Agenda controls and clean startupCraig Jennings4 days3-3/+126
| | | | I restored controlled agenda task mutations and filtered completed items from the dedicated frame. I replaced the treesit-auto revision setter that aborted startup with a runtime slot lookup.
* fix(calendar-sync): guard the hourly timer body against repeating signalsCraig Jennings7 days1-0/+45
| | | | | | | | | - The sync timer body ran unguarded from an hourly run-at-time timer. - A signal in the timezone check or the sync fan-out errored every tick. - The same error, once an hour, indefinitely. - Wrapped the body in condition-case; an error is logged, not propagated. - Also demoted the timezone-change message to the silent log, like the module's other timer-path notices — an hourly timer shouldn't spam the echo area.
* fix(system-defaults): guard and cap the native-comp warning logCraig Jennings7 days1-0/+31
| | | | | | | | - cj/log-comp-warning is :before-until advice on display-warning. - Its append-to-file was unguarded, so an unwritable log path signalled. - That signal propagated out and broke warning display for every comp notice. - Wrapped the write so a failure is swallowed and the warning stays suppressed. - The log also grew without bound; added a 512KB cap that resets it on overflow.
* fix(agenda-frame): keep the allowlist when org-agenda loads firstCraig Jennings7 days2-4/+89
| | | | | | | | | | | | - The read-only shadow walk keeps a key only when the frame map binds it commandp. - It ran from with-eval-after-load partway up the file. - The day/week/redo handlers are defined further down. - With org-agenda already loaded, the walk fired before those defuns existed. - So it read them as non-commands and denied d/w/g/r — the keys the allowlist grants. - That is the normal startup order and every module reload. - Moved the walk to the end of the file, after all the handlers are defined. - A subprocess test loads org-agenda first and pins d/w/g/r surviving, t denied. - Retitled the t-test to assert the deny outcome, not the pre-walk unbound state.
* refactor(org-refile): name the two refile advices so they can be removedCraig Jennings7 days1-0/+87
| | | | | | | | | - Both were anonymous lambdas advising org-refile and org-refile-get-targets. - An anonymous advice can't be advice-removed by reference. - So deleting one from source left a live daemon still running it. - Extracted both to named defuns and installed them remove-then-add. - Now each reloads to a single copy and is removable by reference. - Added unit tests for the target-visiting logic, which had no coverage.
* fix(test): stop the syncing-p tests leaking in-flight stateCraig Jennings7 days2-74/+128
| | | | | | | | | | | | | - They cleared the module's state hash on entry, not on exit. - One marks "proton" syncing to exercise the dispatcher's in-flight guard. - A sibling dispatch test uses a calendar also named "proton". - ERT runs them in that order, so the dispatch became a no-op and it failed. - Now each test let-binds a private hash, matching the sibling test files. - Added a test that pins the isolation itself. The failure was invisible from both directions. make test runs each file in its own Emacs, and the editor hook skipped this family for exceeding its file cap. It only appears when the 63 calendar-sync files run in one process.
* fix(githooks): stop the secret scan passing when it read nothingCraig Jennings7 days2-0/+199
| | | | | | | | | - The scan piped git diff into grep and swallowed failures with || true. - grep exits 1 on no matches, which is normal, so the guard had to stay. - But it also hid a git failure, leaving nothing to scan and reporting clean. - Now the diff is read on its own and a git failure aborts the commit. - The staged-file list feeding the paren check had the same hole. - New bats tests drive a broken git and pin both paths.
* fix: correct four wrong behaviors and drop three dead code pathsCraig Jennings7 days9-411/+526
| | | | | | | | | | | | - yt-dlp downloads claimed completion when tsp had merely queued them. - A fresh machine opened org links in EWW, not Chrome. - ArchWiki search raised a raw error instead of the install hint. - The pre-commit hook kept running when its cd failed. - Removed a second device-grouping implementation nothing called. - Removed a redundant epub advice that ran on every file visit. - Removed a buffer-burying clause naming a mode Emacs doesn't define. - Moved the Hugo bindings onto the custom prefix map. - The ArchWiki docs path is now a variable, making it testable.
* fix(auto-dim): stop the minibuffer from re-dimming the window you leftCraig Jennings8 days1-1/+6
| | | | | | | - Opening the minibuffer dimmed whatever window you had just left. - Dimming follows selection, and the minibuffer takes selection. - auto-dim-other-buffers-dim-on-switch-to-minibuffer sat at its package default of t. - Set it to nil, so minibuffer entry leaves the dim state alone.
* fix: make the Full Agenda frame's default-deny actually denyCraig Jennings10 days1-0/+28
| | | | | | The frame's read-only policy leaned on a [t] catch-all, but a keymap's default binding never shadows an explicit binding in a lower-priority map. So every key org-agenda-mode-map binds sailed through the catch-all and could mutate source files from the "read-only" frame: t (todo), I (clock-in), k (capture), z (add-note), s (save-all), the C-c schedule/deadline/clock mutators, C-x C-s. Walk org-agenda-mode-map recursively on load and explicitly deny every sequence not on the frame's allowlist, so the catch-all's intent holds. Navigation, engage, refresh, d/w, and C-c C-o still work, and M-x is unaffected.
* feat(music): move single to 1, save to s, drop the stop keyCraig Jennings10 days2-3/+21
| | | | In the playlist buffer, single-track mode moves off s onto 1 and save moves off v onto s. The stop key (S) is gone, since SPC/pause already covers stopping. The Mode/Keys header legend follows, and v is now unbound. Stop is still reachable from the global music prefix (C-; m s).
* feat: add d/w day/week view toggle to the Full Agenda frameCraig Jennings10 days1-4/+63
| | | | The frame was locked to the 7-day span, with d and w denied as view-changes. The span is now a variable the F command reads. Org re-evaluates custom-command settings on every build and redo, so a day view survives the 5-minute refresh tick. d shows today's single day, w restores the 7-day view, and each spawn resets to 7 so a fresh frame opens at the default. The remaining date-navigation keys (y/f/b/j) stay denied to keep the frame today-anchored.
* fix: stop M-SPC / M-S-SPC escaping the Full Agenda frameCraig Jennings10 days1-0/+10
| | | | M-SPC and M-S-SPC (the ai-term agent-swap chords) are bound in the global map, so pressing them inside the read-only frame jumped to ai-term instead of being contained. The frame's [t] catch-all can't deny them: a keymap's default binding doesn't shadow an explicit binding in a lower-priority map. Deny both explicitly in the frame keymap.
* fix: bind g to refresh inside the Full Agenda frameCraig Jennings10 days1-3/+6
| | | | The frame's default-deny policy classed g as a view-change and denied it, so the muscle-memory refresh key printed the fixed-view message instead of rebuilding. Bind g to the same frame-scoped safe redo as r. C-M-<f8> stays the force-rescan.
* refactor(elfeed): remove dead cj/extract-stream-urlCraig Jennings11 days1-38/+1
| | | | No production caller existed, and the v binding already plays entries through mpv, which resolves YouTube pages via its own yt-dlp hook internally, so the manual stream-URL extraction was fully superseded. Its tests go with it; the process-entries coverage stays.
* fix(system): resolve the locker at command time, not module loadCraig Jennings11 days1-0/+32
| | | | lockscreen-cmd baked (env-wayland-p) into a defvar at load, so a daemon started before WAYLAND_DISPLAY reached its environment froze the locker to slock and Lock failed silently on Wayland. The lock command now resolves per call, with lockscreen-cmd kept as an explicit override. Craig confirmed the fix over the decline option since it also makes the systemd unit path safe to revive.
* test: make the 31st-recurrence and immediate-insert tests env-stableCraig Jennings11 days2-8/+18
| | | | The calendar test's range ended at midnight Dec 31, excluding that day's occurrence, and the org-roam test asserted special-variable-p, which answers differently in the package-loaded hook env versus bare batch (a bare defvar is a file-local declaration). The recurrence test now ends the range past the year and the insert test asserts the binding behaviorally.
* perf(ai-term): fetch the tmux session list once per launchCraig Jennings11 days1-0/+40
| | | | The launch path spawned tmux list-sessions up to three times: in the project picker's sorting, in the launcher's fresh check, and again inside show-or-create. The list is now fetched once and threaded through; the cycling path still fetches its own.
* perf(host-env): try cheap timezone methods before the zoneinfo scanCraig Jennings11 days1-4/+15
| | | | The content-comparison scan reads hundreds of zoneinfo files and ran first on every call. TZ, /etc/timezone, and the symlink target are O(1) and answer on almost every system; the scan is now the last resort for a copied /etc/localtime. Verified the symlink and scan agree on this machine.
* refactor(comments): extract shared comment-syntax resolutionCraig Jennings11 days1-0/+25
| | | | The comment-start/comment-end resolution was copied into six command wrappers. One cj/--comment-read-syntax now owns it; the block-banner wrapper keeps its own block-specific prompt.
* fix(dev-fkeys): run javascript test files through the npx runnerCraig Jennings11 days1-4/+9
| | | | The language detector classifies js/jsx and the test-file detector recognizes JS tests, but the dispatch had no javascript arm, so C-F6 on a JS test errored though the typescript npx path runs it fine. javascript now shares that arm.
* fix(calendar): skip short months for plain monthly-on-the-31st rulesCraig Jennings11 days1-0/+18
| | | | The stepper kept day-of-month verbatim, so Jan 31 stepped to Feb 31 and encode-time normalized it into a phantom Mar 3 occurrence. Plain monthly rules now advance to the next month that has the day, per RFC 5545.
* fix(comments): render padded dividers at LENGTH exactlyCraig Jennings11 days1-0/+18
| | | | available-width omitted the doubled semicolon and the space after comment-start that the emit path adds, so dividers rendered LENGTH+2 (elisp) or LENGTH+1 wide. The budget now mirrors every term min-length already counts.
* fix(music): queue all dired-marked files and de-anonymize header adviceCraig Jennings11 days2-0/+89
| | | | | | cj/music-add-dired-selection gated dired-get-marked-files behind use-region-p, so files marked with m fell to the single-file branch and only the point file was queued. dired-get-marked-files already honors marks, a region, or point on its own. The header-refresh advice on the four EMMS toggles was an anonymous lambda, which can't be advice-removed and stacked a copy on every :config reload. It's now a named function installed remove-then-add.
* fix(undead-buffers): keep C-u on wrapper commands from spamming the undead listCraig Jennings11 days1-0/+17
| | | | The delegated cj/kill-buffer-or-bury-alive reads current-prefix-arg, so a prefixed wrapper call took the add-to-undead-list branch for every buffer: nothing killed, list spammed. The four wrappers now bind the prefix to nil around the delegate.
* fix(dashboard): make the bookmarks override realCraig Jennings11 days1-0/+10
| | | | The top-level defun was clobbered when use-package reloaded dashboard-widgets, so the override never ran. Deferring it kept it alive but exposed a second bug: dashboard-insert-section is a macro, unknown at module compile time, so the body compiled as a function call and crashed void-variable at render. The override now registers after dashboard-widgets and compiles with the macro known.
* fix(org-roam): declare capture-templates special for immediate insertCraig Jennings11 days1-0/+8
| | | | Without the defvar, the byte-compiled let-binding in cj/org-roam-node-insert-immediate is a dead lexical binding and :immediate-finish never reaches org-roam-node-insert, so the immediate insert opened a capture buffer.
* fix(jumper): handle empty completing-read input without crashingCraig Jennings11 days1-0/+25
| | | | An unmatched choice made the assoc lookup nil, which crashed the index arithmetic with wrong-type-argument. The jump prompt now gives a user-error and the remove prompt cancels.
* fix(external-open): guard looping video open behind executable-findCraig Jennings11 days1-0/+10
| | | | The command fires via the find-file advice, so visiting a video without mpv on PATH threw an opaque call-process error mid-visit.
* fix(slack): friendly error on reaction command before slack loadsCraig Jennings11 days1-0/+8
| | | | slack-current-buffer is declared but void until slack.el loads, so a cold C-; S ! signaled void-variable instead of the not-in-a-Slack-buffer message.
* fix(flyspell): escape the + so org src blocks are actually skippedCraig Jennings11 days1-0/+18
| | | | The skip entry used "#+" (one-or-more #), which matches no real begin_src line, so ispell spell-checked inside every org code block.
* feat(agenda): suspend auto-dim while the agenda frame is openCraig Jennings11 days1-0/+70
| | | | The refresh tick's selection swing marks the working window non-selected, and auto-dim's debounced dim lands after the tick, so the working frame visibly dimmed every five minutes. Spawn now turns auto-dim-other-buffers-mode off (remembering it was on) and closing the frame restores it. A spawn with auto-dim already off touches nothing. It also keeps the standing agenda fully lit instead of dimmed-when-unfocused.
* fix(agenda): hide the refresh tick's selection flickerCraig Jennings11 days1-0/+32
| | | | The rebuild takes a second or two, and for its whole duration the agenda window was the selected window, so the cursor visibly went hollow in the working frame on every tick. It read as focus theft, though no focus function ever ran. The tick now inhibits redisplay (the synchronous rebuild blocks Emacs anyway, so this costs nothing) and restores selection before redisplay resumes. A tick that would fire during an active minibuffer is skipped.
* fix(agenda): repair the refresh tick, tighten the gutter, allow point motionCraig Jennings11 days1-2/+47
| | | | | | | | Every 5-minute refresh failed. The F command's general settings carried (org-agenda-sticky t), and those settings are baked into the buffer's series-redo-cmd and re-applied on each redo. With the buffer already existing, org-agenda-use-sticky-p turned true mid-redo and org-agenda-prepare threw 'exit ("use r to refresh") to a catch that doesn't exist inside org-agenda-redo. I removed sticky from the command settings (the spawn wrapper still binds it, which is where the buffer gets its name) and kept the redo path free of sticky, since org-agenda-redo handles the in-place rebuild itself. Verified with a real tick against a live frame: rebuild succeeds, the failure banner clears, the latch resets. The wide gutter between the source column and the item was the global agenda prefix format's 25-char category pad. The F view now sets its own format with a 10-char pad. C-a hit the deny catch-all. I allowlisted read-only point motion (C-a, C-e, C-f, C-b) and isearch (C-s, C-r).
* fix(agenda): keep the agenda frame a single full-height agenda windowCraig Jennings11 days1-1/+40
| | | | | | | | The frame spawned with the working buffer in its top quarter and the agenda below at 75%. The sticky setting inside the F custom command applies too late to name the buffer, so it rendered as plain *Org Agenda*, and that name matches the 0.75 below-selected display rule in org-agenda-config.el, which split the brand-new frame. I bind org-agenda-sticky and current-window dynamically around the render, so the buffer is *Org Agenda(F)* (no display rule matches it) and it takes the frame's sole window. A delete-other-windows after the render is the belt in case any future rule still splits. The frame is also named "Full Agenda" now so it's recognizable at a glance and WM rules can target it.
* fix(agenda): make C-x C-c close the agenda frame, not the daemonCraig Jennings11 days1-0/+8
| | | | The catch-all denied it before, and letting the global through would be worse: save-buffers-kill-terminal on a make-frame frame has no client to close, so it kills the daemon. Inside the frame the gesture now runs the frame close.
* fix(agenda): stop the frame's catch-all from swallowing input machineryCraig Jennings11 days1-3/+62
| | | | | | | | The default-deny [t] binding intercepted far more than Org commands. switch-frame events hit the deny handler, so every focus change into or out of the frame spammed the read-only message and broke frame switching. Mouse wheel, mouse-1 clicks, and the help prefix were dead. And S-<f8>/C-M-<f8> were never bound in the mode map, so the frame's own toggle couldn't close it from inside. I punched the machinery through the catch-all (an explicit nil shadows the [t] default in the same map, so those events fall through to their global bindings) and bound the two frame controls per the spec's allowlist. Two redo-lifecycle bugs rode along, both from org-agenda-mode's kill-all-local-variables: the buffer-local kill-buffer-hook was stripped on the first refresh tick (killing the buffer would orphan the frame), and the failure banner was held in a buffer-local var while its overlay survives erase-buffer, so a banner could never be removed after a later success. The finalize re-enable now re-adds the hook, and the banner is tagged with an overlay property and found by scanning.
* fix(agenda): spawn the agenda frame tiled, not compositor-fullscreenCraig Jennings11 days1-0/+10
| | | | I dropped (fullscreen . fullboth) from the spawned frame so a tiling WM places it side by side with the working frame. "Fullscreen" was the wrong word for the intent: a normal frame at its full tiled position, not one covering the whole output. The engage-routing and focus logic are unchanged and matter more now, since a task opens in the adjacent working pane.
* feat(agenda): add S-<f8> dedicated fullscreen agenda frameCraig Jennings11 days1-0/+736
| | | | | | | | I added a standing agenda surface in its own fullscreen frame of the daemon, placeable on its own workspace while the working frames stay untouched. S-<f8> spawns, raises, or closes it. The force-rescan moves to C-M-<f8>. It shows a today-anchored seven-day view and refreshes every five minutes through org-agenda-redo, so the now-line and synced events stay current without re-scanning the file list. A default-deny minor mode keeps it read-only: only navigation, the engage keys (routed to the working frame), and the frame's own controls are allowed. A failed redo falls back to the last-good snapshot with cloned markers, so the frame is never blank or unrestricted. I made it a frame of the daemon rather than a second process, so it shares the live state (calendar-sync, edits, the now-line). There's no startup auto-open, and the Hyprland-managed-window variant is a vNext once this proves out.
* feat(fontaine): rename Coding to Coding M, add Coding L at 14ptCraig Jennings11 days1-11/+13
| | | | | The Berkeley Mono code ladder is now Coding XS (11pt), Coding M (13pt), Coding L (14pt), and Coding XL (16pt).
* feat(ai-term): M-SPC cycles attached agents, M-S-SPC cycles allCraig Jennings11 days2-6/+68
| | | | | | | | | | | | M-SPC now steps only through attached agents (the ones with a live on-screen buffer), so the fast chord stays among what's already up. M-S-SPC steps through all active agents and attaches a detached tmux session when it lands on one, which is what M-SPC used to do. I moved the shared step logic into cj/--ai-term-step-among and added cj/--ai-term-attached-agent-dirs to filter the queue to attached buffers. I bound both chords globally and in eat-semi-char-mode-map so they work from inside an agent buffer. C-; a n keeps cycling all.
* feat: add workflow font profilesCraig Jennings12 days3-105/+255
| | | | | | I replaced the mixed Fontaine presets with seven named profiles that persist across restarts. The mode line, echo area, and minibuffer stay in Berkeley Mono. Nov applies the shared Reading profile inside each EPUB buffer without changing the global selection.
* fix(shell): require external-open, scope tramp revert, drop dead configCraig Jennings13 days2-9/+39
| | | | | | | | - dirvish-config, dwim-shell-config: both call cj/xdg-open (external-open) but relied on init.el requiring it before them, and dirvish's require comment misattributed it to system-utils. Added the require to both and corrected the comment and both Runtime-requires headers. - tramp-config: revert-without-query was set to '(".*") inside the TRAMP module, so every file in Emacs reverted without confirmation. Scoped it to the TRAMP path regexp, so local files keep their revert prompt and remote files skip the round-trip. - tramp-config: removed the sshfast method (referenced nowhere, scp and sshx handle everything) and a duplicate tramp-own-remote-path add-to-list. Tests: new runtime-requires assertions for both modules, red first under make test-file. The tramp :config edits are verified in the daemon. Suite green.