feat(org-noter): implement custom org-noter workflow

Implemented custom org-noter workflow with F6 keybinding:
- Creates notes files as org-roam nodes in org-roam-directory
- Title prompt with pre-slugified default, notes-on-{slug}.org format
- F6 toggles notes window visibility when session active
- Preserves PDF fit setting on toggle
- Deferred org-roam integration to prevent PDF open hang

Also fixed: quick-sdcv quit binding, calendar-sync sentinel buffer error
Added 30 ERT tests for title-to-slug and template generation functions

1 files changed, 109 insertions, 0 deletions

diff --git a/tests/test-org-noter--generate-notes-template.el b/tests/test-org-noter--generate-notes-template.el
new file mode 100644
index 00000000..df545ccf
--- /dev/null
+++ b/tests/test-org-noter--generate-notes-template.el
@@ -0,0 +1,109 @@
+;;; test-org-noter--generate-notes-template.el --- Tests for cj/org-noter--generate-notes-template -*- lexical-binding: t; -*-
+
+;;; Commentary:
+;; ERT tests for the generate-notes-template function used in org-noter workflow.
+;; Tests cover normal, boundary, and error cases.
+
+;;; Code:
+
+(require 'ert)
+(require 'org-noter-config)
+
+;;; Test Helpers
+
+(defun test-org-noter--template-has-property (template property value)
+  "Check if TEMPLATE contains PROPERTY with VALUE in properties drawer."
+  (string-match-p (format ":%s: %s" property (regexp-quote value)) template))
+
+(defun test-org-noter--template-has-keyword (template keyword value)
+  "Check if TEMPLATE contains #+KEYWORD: VALUE."
+  (string-match-p (format "#\\+%s: %s" keyword (regexp-quote value)) template))
+
+;;; Normal Cases
+
+(ert-deftest test-org-noter--generate-notes-template-normal-basic ()
+  "Normal case: Basic template generation."
+  (let ((template (cj/org-noter--generate-notes-template "Test Book" "/path/to/book.pdf")))
+    (should (stringp template))
+    (should (string-match-p ":PROPERTIES:" template))
+    (should (string-match-p ":END:" template))
+    (should (string-match-p "\\* Notes" template))))
+
+(ert-deftest test-org-noter--generate-notes-template-normal-has-id ()
+  "Normal case: Template has ID property."
+  (let ((template (cj/org-noter--generate-notes-template "Test Book" "/path/to/book.pdf")))
+    (should (string-match-p ":ID: [a-f0-9-]+" template))))
+
+(ert-deftest test-org-noter--generate-notes-template-normal-has-noter-document ()
+  "Normal case: Template has NOTER_DOCUMENT property."
+  (let ((template (cj/org-noter--generate-notes-template "Test Book" "/path/to/book.pdf")))
+    (should (test-org-noter--template-has-property template "NOTER_DOCUMENT" "/path/to/book.pdf"))))
+
+(ert-deftest test-org-noter--generate-notes-template-normal-has-roam-refs ()
+  "Normal case: Template has ROAM_REFS property."
+  (let ((template (cj/org-noter--generate-notes-template "Test Book" "/path/to/book.pdf")))
+    (should (test-org-noter--template-has-property template "ROAM_REFS" "/path/to/book.pdf"))))
+
+(ert-deftest test-org-noter--generate-notes-template-normal-has-title ()
+  "Normal case: Template has title with book name."
+  (let ((template (cj/org-noter--generate-notes-template "The Great Gatsby" "/books/gatsby.epub")))
+    (should (test-org-noter--template-has-keyword template "title" "Notes on The Great Gatsby"))))
+
+(ert-deftest test-org-noter--generate-notes-template-normal-has-filetags ()
+  "Normal case: Template has ReadingNotes filetag."
+  (let ((template (cj/org-noter--generate-notes-template "Test Book" "/path/to/book.pdf")))
+    (should (test-org-noter--template-has-keyword template "FILETAGS" ":ReadingNotes:"))))
+
+(ert-deftest test-org-noter--generate-notes-template-normal-has-category ()
+  "Normal case: Template has CATEGORY set to book title."
+  (let ((template (cj/org-noter--generate-notes-template "Clean Code" "/books/clean-code.pdf")))
+    (should (test-org-noter--template-has-keyword template "CATEGORY" "Clean Code"))))
+
+;;; Boundary Cases
+
+(ert-deftest test-org-noter--generate-notes-template-boundary-long-title ()
+  "Boundary case: Very long title."
+  (let* ((long-title "This Is An Incredibly Long Book Title That Goes On And On")
+         (template (cj/org-noter--generate-notes-template long-title "/books/long.pdf")))
+    (should (test-org-noter--template-has-keyword template "title" (format "Notes on %s" long-title)))
+    (should (test-org-noter--template-has-keyword template "CATEGORY" long-title))))
+
+(ert-deftest test-org-noter--generate-notes-template-boundary-special-chars-in-title ()
+  "Boundary case: Special characters in title."
+  (let ((template (cj/org-noter--generate-notes-template "C++: A Guide" "/books/cpp.pdf")))
+    (should (test-org-noter--template-has-keyword template "title" "Notes on C++: A Guide"))))
+
+(ert-deftest test-org-noter--generate-notes-template-boundary-special-chars-in-path ()
+  "Boundary case: Special characters in path."
+  (let ((template (cj/org-noter--generate-notes-template "Test" "/path/with spaces/book.pdf")))
+    (should (test-org-noter--template-has-property template "NOTER_DOCUMENT" "/path/with spaces/book.pdf"))))
+
+(ert-deftest test-org-noter--generate-notes-template-boundary-epub-path ()
+  "Boundary case: EPUB file path."
+  (let ((template (cj/org-noter--generate-notes-template "Novel" "/library/novel.epub")))
+    (should (test-org-noter--template-has-property template "NOTER_DOCUMENT" "/library/novel.epub"))))
+
+;;; Structure Tests
+
+(ert-deftest test-org-noter--generate-notes-template-structure-properties-first ()
+  "Structure: Properties drawer comes first."
+  (let ((template (cj/org-noter--generate-notes-template "Test" "/path.pdf")))
+    (should (string-match "\\`:PROPERTIES:" template))))
+
+(ert-deftest test-org-noter--generate-notes-template-structure-notes-heading ()
+  "Structure: Has Notes heading for content."
+  (let ((template (cj/org-noter--generate-notes-template "Test" "/path.pdf")))
+    (should (string-match-p "^\\* Notes$" template))))
+
+(ert-deftest test-org-noter--generate-notes-template-structure-unique-ids ()
+  "Structure: Each call generates unique ID."
+  (let ((template1 (cj/org-noter--generate-notes-template "Test1" "/path1.pdf"))
+        (template2 (cj/org-noter--generate-notes-template "Test2" "/path2.pdf")))
+    (string-match ":ID: \\([a-f0-9-]+\\)" template1)
+    (let ((id1 (match-string 1 template1)))
+      (string-match ":ID: \\([a-f0-9-]+\\)" template2)
+      (let ((id2 (match-string 1 template2)))
+        (should-not (equal id1 id2))))))
+
+(provide 'test-org-noter--generate-notes-template)
+;;; test-org-noter--generate-notes-template.el ends here