org-drill/tests/test-org-drill-resume-nil-session.el, branch main Spaced-repetition flashcards for Org Mode https://git.cjennings.net/org-drill/atom?h=main 2026-05-05T19:35:45+00:00 test: cover --pick-next-marker and resume happy-paths 2026-05-05T19:35:45+00:00 Craig Jennings c@cjennings.net 2026-05-05T19:35:45+00:00 urn:sha1:df4caddc17091b67b5afa7864b53ccffc00ce076 I extended `tests/test-org-drill-route-rating-result.el' with four `org-drill--pick-next-marker' cases (no resume → pop, resume with live drill marker → keep current-item and clear resume-p, resume with nil or non-drill current-item → fall through to fresh pop). I also extended the resume regression file with the three happy-path branches of `org-drill-resume': pending entries → resume, finished with pending count → y-or-n-p offers a new session, finished with nothing → print 'finished'. Coverage moved from 91.7% to 92.1%. fix: guard org-drill-again and org-drill-resume against nil last-session 2026-05-05T10:16:39+00:00 Craig Jennings c@cjennings.net 2026-05-05T10:16:39+00:00 urn:sha1:f5d67d7a0a48cd68d7f443ca475c0c7bc59a8468 Both functions bound session to org-drill-last-session and immediately called setf / org-drill-entries-pending-p on it without checking for nil. First-time invocation (or after Emacs restart with no active session) threw an obscure eieio-oset / nil-slot type error instead of a clear message. Added (unless session (user-error ...)) at the top of each function. A user running M-x org-drill-resume cold now sees a sensible message telling them to run org-drill first.