<feed xmlns='http://www.w3.org/2005/Atom'>
<title>org-drill/tests/test-org-drill-explain-and-language-cards.el, branch main</title>
<subtitle>Spaced-repetition flashcards for Org Mode
</subtitle>
<id>https://git.cjennings.net/org-drill/atom?h=main</id>
<link rel='self' href='https://git.cjennings.net/org-drill/atom?h=main'/>
<link rel='alternate' type='text/html' href='https://git.cjennings.net/org-drill/'/>
<updated>2026-05-27T00:58:21+00:00</updated>
<entry>
<title>refactor: dedupe presenters, group defcustoms, and fill in docstrings</title>
<updated>2026-05-27T00:58:21+00:00</updated>
<author>
<name>Craig Jennings</name>
<email>c@cjennings.net</email>
</author>
<published>2026-05-27T00:58:21+00:00</published>
<link rel='alternate' type='text/html' href='https://git.cjennings.net/org-drill/commit/?id=7eece407772d7c5cfba93ba914439094f0d9fbf2'/>
<id>urn:sha1:7eece407772d7c5cfba93ba914439094f0d9fbf2</id>
<content type='text'>
A cleanup pass over org-drill internals, squashed from the refactor/wave3-cleanup branch. No behavior change. Each step kept the existing tests green and added its own.

I shared two duplicated helpers across the language card getters: org-drill--read-property-string and org-drill--face-from-alist.

I factored the cloze body-scan out of the two multicloze presenters into org-drill--cloze-body-bounds, org-drill--count-cloze-matches, and org-drill--hide-cloze-by-index, so each presenter just picks which indices to hide.

I pulled the presenter resolution and the four-way result classification out of org-drill-entry-f into org-drill--resolve-presenter and org-drill--classify-presentation-result, untangling the pivot of every drill iteration.

I split the 37 defcustoms (and the three cloze faces) into four customize sub-groups (display, algorithm, session, leech) so customize-group org-drill is navigable. There's no leitner group because the Leitner settings are defvars.

I documented the 22 defuns that had no docstring, rewrote the corrupted org-drill-presentation-prompt-in-mini-buffer docstring, and switched eleven docstrings to the imperative "Return" (issue #2).
</content>
</entry>
<entry>
<title>test: add explain-entry-p, end-of-entry-pos, and language card info coverage</title>
<updated>2026-05-05T09:23:37+00:00</updated>
<author>
<name>Craig Jennings</name>
<email>c@cjennings.net</email>
</author>
<published>2026-05-05T09:23:37+00:00</published>
<link rel='alternate' type='text/html' href='https://git.cjennings.net/org-drill/commit/?id=0ce03604b5267f301c119ed75e0eba9b5104c6b9'/>
<id>urn:sha1:0ce03604b5267f301c119ed75e0eba9b5104c6b9</id>
<content type='text'>
14 ERT tests covering:

- org-drill-explain-entry-p: with/without :explain: tag, no-inherit
  flag rejects parent's tag
- org-drill-end-of-entry-pos: single-heading and multi-heading subtree
  bounds
- org-drill-get-verb-conjugation-info: full property read, tense-only
  (mood optional), missing-required errors, tense-color highlight face
- org-drill-get-noun-info: full property read, missing-required errors,
  feminine-gender orchid color from alist, unknown-gender red fallback
</content>
</entry>
</feed>
