diff options
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/test-pearl-save.el | 31 |
1 files changed, 31 insertions, 0 deletions
diff --git a/tests/test-pearl-save.el b/tests/test-pearl-save.el index 65f742a..8050e6d 100644 --- a/tests/test-pearl-save.el +++ b/tests/test-pearl-save.el @@ -375,6 +375,37 @@ the conflict choice (stubbed). Returns (:outcome :fetched :pushed :applied (should (eq 'failed (plist-get r :status))) (should (eq 'fetch-failed (plist-get r :reason))))) +(ert-deftest test-pearl-atomic-killed-buffer-no-signal () + "A push whose buffer is killed before the callback reports without touching it. +The commit advances the baseline via the marker; on a dead buffer it must be +skipped (the remote push already landed), not signal." + (let ((buf (generate-new-buffer "pearl-atomic-kill")) outcome committed) + (unwind-protect + (let (marker) + (with-current-buffer buf + (insert "* h\n:PROPERTIES:\n:LINEAR-ID: a\n:END:\n") + (org-mode) + (goto-char (point-min)) + (setq marker (point-marker))) + (pearl--run-atomic-field-save + (list :issue-id "u" :identifier "ENG-1" :field 'assignee :marker marker + :label "ENG-1 assignee" :live "b" :baseline "a" :equal #'string= + :live-display "b" :remote-display (lambda (_) "a") + ;; remote = baseline -> clean push; the fetch kills the buffer first + :fetch-remote (lambda (cb) (kill-buffer buf) (funcall cb "a")) + :push (lambda (cb) (funcall cb '(:success t))) + ;; touching the dead marker would signal without the guard + :commit-local (lambda () + (setq committed t) + (org-entry-put marker "X" "y")) + :commit-remote (lambda (_) nil)) + (lambda (o) (setq outcome o))) + (should-not (buffer-live-p buf)) + ;; remote success is still reported; the local commit is skipped + (should (eq 'pushed (plist-get outcome :status))) + (should-not committed)) + (when (buffer-live-p buf) (kill-buffer buf))))) + ;;; --save-assignee-field (the relation saver, end to end) (ert-deftest test-pearl-save-assignee-clean-push-advances-baseline () |