Earl Chase <[email protected]> writes: > I added more tests for week and month repeaters. > Previously, `test-org/auto-repeat-maybe' only had tests for year, day and > hour repeaters.
More comments inline :) > + (let* ((headline (org-element-lineage > + (org-element-at-point) > + 'headline t)) You missed inlinetasks. > + (ts (match-string 0)) Do you really still need TS? Can't you reuse the timestamp object? > + (cl-macrolet ((test-auto-repeat-maybe (buffer-text test-time expected) > + (let ((org-todo-keywords '((sequence "TODO" "DONE")))) > + `(org-test-with-temp-text ,buffer-text Why not `(let ...)? > + (org-test-at-time ,test-time > + (org-todo "DONE") > + (should (string-match-p (regexp-quote ,expected) > (buffer-string)))))))) > + "Test `org-auto-repeat-maybe' specifications." Looks like docstring got displaced. -- Ihor Radchenko // yantar92, Org mode maintainer, Learn more about Org mode at <https://orgmode.org/>. Support Org development at <https://liberapay.com/org-mode>, or support my work at <https://liberapay.com/yantar92>
