aboutsummaryrefslogtreecommitdiff
path: root/archive/gptel/tests/test-gptel-tools-git-diff.el
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2026-06-23 20:12:58 -0400
committerCraig Jennings <c@cjennings.net>2026-06-23 20:12:58 -0400
commite41c25068d0cec9434895a6d3e3a25d3a26f645f (patch)
tree5e30938a3fd6d80f501ffe3e6c1c187c5ddeb2c9 /archive/gptel/tests/test-gptel-tools-git-diff.el
parenta936e081b7270fbd4f1e7e9cb67ca1d4c2291ce6 (diff)
downloaddotemacs-e41c25068d0cec9434895a6d3e3a25d3a26f645f.tar.gz
dotemacs-e41c25068d0cec9434895a6d3e3a25d3a26f645f.zip
chore(ai): archive gptel and remove it from the live config
I archived gptel to archive/gptel/ since I rarely use it. Moved there: the six gptel modules (ai-config, ai-conversations, ai-conversations-browser, ai-mcp, ai-quick-ask, ai-rewrite), the gptel-tools/ directory, custom/gptel-prompts.el, their test files and utilities, and the four gptel-only specs. Scrubbed from the live config: the ai-config require in init.el, which also drops the whole C-; a keymap; the gptel-mode emojify hook in font-config.el; the gptel-tools entries in the Makefile clean target and the coverage runner; and the gptel feature notes in README. Cancelled the open gptel tasks in todo.org (the AI Open Work issues, the feature-extension brainstorm, the velox gptel-magit bug). ai-term stays. It is the ghostel Claude launcher, independent of gptel. Verified: every module loads, a batch init launch reaches completion clean, and the full test suite shows only pre-existing coverage failures unrelated to this change.
Diffstat (limited to 'archive/gptel/tests/test-gptel-tools-git-diff.el')
-rw-r--r--archive/gptel/tests/test-gptel-tools-git-diff.el163
1 files changed, 163 insertions, 0 deletions
diff --git a/archive/gptel/tests/test-gptel-tools-git-diff.el b/archive/gptel/tests/test-gptel-tools-git-diff.el
new file mode 100644
index 000000000..114fec293
--- /dev/null
+++ b/archive/gptel/tests/test-gptel-tools-git-diff.el
@@ -0,0 +1,163 @@
+;;; test-gptel-tools-git-diff.el --- Tests for git_diff gptel tool -*- lexical-binding: t; -*-
+
+;;; Commentary:
+;; Tests run against real temp git repos under HOME via `process-file'.
+
+;;; Code:
+
+(require 'ert)
+(require 'cl-lib)
+
+(eval-and-compile
+ (add-to-list 'load-path (expand-file-name "tests" user-emacs-directory))
+ (add-to-list 'load-path (expand-file-name "gptel-tools" user-emacs-directory))
+ (setq load-prefer-newer t)
+ (unless (featurep 'gptel)
+ (defvar gptel-tools nil)
+ (defun gptel-make-tool (&rest _args) nil)
+ (defun gptel-get-tool (&rest _args) nil)
+ (provide 'gptel)))
+
+(require 'git_diff)
+
+;; ---------- helpers
+
+(defun test-gptel-tools-git-diff--with-repo (fn)
+ "Create a temp git repo under HOME with one committed file, call FN."
+ (let* ((name (format ".test-gptel-tools-git-diff-%s"
+ (format-time-string "%s%N")))
+ (dir (expand-file-name name "~")))
+ (unwind-protect
+ (progn
+ (make-directory dir)
+ (let ((default-directory dir))
+ (call-process "git" nil nil nil "init" "--quiet")
+ (call-process "git" nil nil nil "config" "user.email" "test@x")
+ (call-process "git" nil nil nil "config" "user.name" "Test")
+ (with-temp-file (expand-file-name "f.txt" dir)
+ (insert "original\n"))
+ (call-process "git" nil nil nil "add" "f.txt")
+ (call-process "git" nil nil nil "commit" "--quiet" "-m" "initial"))
+ (funcall fn dir))
+ (when (file-exists-p dir) (delete-directory dir t)))))
+
+;; ---------- build-args
+
+(ert-deftest test-gptel-tools-git-diff-build-args-no-refs ()
+ "Normal: no refs / no file → bare diff args."
+ (should (equal (cj/gptel-git-diff--build-args nil nil nil)
+ '("-c" "color.ui=false" "diff"))))
+
+(ert-deftest test-gptel-tools-git-diff-build-args-with-ref1 ()
+ "Normal: REF1 appended."
+ (should (equal (cj/gptel-git-diff--build-args "HEAD~1" nil nil)
+ '("-c" "color.ui=false" "diff" "HEAD~1"))))
+
+(ert-deftest test-gptel-tools-git-diff-build-args-with-both-refs ()
+ "Normal: REF1 and REF2 both appended."
+ (should (equal (cj/gptel-git-diff--build-args "HEAD~1" "HEAD" nil)
+ '("-c" "color.ui=false" "diff" "HEAD~1" "HEAD"))))
+
+(ert-deftest test-gptel-tools-git-diff-build-args-with-file ()
+ "Normal: FILE appended after `--'."
+ (should (equal (cj/gptel-git-diff--build-args nil nil "foo.txt")
+ '("-c" "color.ui=false" "diff" "--" "foo.txt"))))
+
+(ert-deftest test-gptel-tools-git-diff-build-args-boundary-empty-strings ()
+ "Boundary: empty-string REF/FILE values are ignored."
+ (should (equal (cj/gptel-git-diff--build-args "" "" "")
+ '("-c" "color.ui=false" "diff"))))
+
+;; ---------- truncate
+
+(ert-deftest test-gptel-tools-git-diff-truncate-under-cap ()
+ "Normal: short input returns unchanged."
+ (should (equal (cj/gptel-git-diff--truncate "small diff") "small diff")))
+
+(ert-deftest test-gptel-tools-git-diff-truncate-over-cap ()
+ "Boundary: output exceeding the cap is truncated with a marker."
+ (let* ((cap cj/gptel-git-diff--max-output-bytes)
+ (huge (make-string (+ cap 1000) ?x))
+ (out (cj/gptel-git-diff--truncate huge)))
+ (should (string-match-p "\\[truncated:" out))
+ (should (> (length huge) (length out)))))
+
+;; ---------- validate-path
+
+(ert-deftest test-gptel-tools-git-diff-validate-path-normal ()
+ "Normal: validator accepts a git working tree."
+ (test-gptel-tools-git-diff--with-repo
+ (lambda (dir)
+ (should (equal (cj/gptel-git-diff--validate-path dir) dir)))))
+
+(ert-deftest test-gptel-tools-git-diff-validate-path-error-outside-home ()
+ "Error: path outside HOME signals."
+ (should-error (cj/gptel-git-diff--validate-path "/etc")))
+
+(ert-deftest test-gptel-tools-git-diff-validate-path-error-not-a-repo ()
+ "Error: non-git directory signals."
+ (let ((dir (make-temp-file
+ (expand-file-name ".test-gptel-tools-git-diff-" "~") t)))
+ (unwind-protect
+ (should-error (cj/gptel-git-diff--validate-path dir))
+ (when (file-exists-p dir) (delete-directory dir t)))))
+
+(ert-deftest test-gptel-tools-git-diff-validate-path-error-not-a-directory ()
+ "Error: file paths are rejected."
+ (let ((file (make-temp-file
+ (expand-file-name ".test-gptel-tools-git-diff-file-" "~"))))
+ (unwind-protect
+ (should-error (cj/gptel-git-diff--validate-path file))
+ (when (file-exists-p file) (delete-file file)))))
+
+(ert-deftest test-gptel-tools-git-diff-validate-path-error-symlink-outside-home ()
+ "Error: symlinked directories resolving outside HOME are rejected."
+ (let ((link (expand-file-name
+ (format ".test-gptel-tools-git-diff-link-%s"
+ (format-time-string "%s%N"))
+ "~")))
+ (unwind-protect
+ (progn
+ (make-symbolic-link "/tmp" link t)
+ (should-error (cj/gptel-git-diff--validate-path link)))
+ (when (file-symlink-p link) (delete-file link)))))
+
+;; ---------- run
+
+(ert-deftest test-gptel-tools-git-diff-run-no-changes ()
+ "Boundary: a clean tree with no refs returns the no-diff marker."
+ (test-gptel-tools-git-diff--with-repo
+ (lambda (dir)
+ (let ((out (cj/gptel-git-diff--run dir)))
+ (should (string-match-p "No diff" out))))))
+
+(ert-deftest test-gptel-tools-git-diff-run-unstaged-change ()
+ "Normal: an unstaged edit appears as a real diff."
+ (test-gptel-tools-git-diff--with-repo
+ (lambda (dir)
+ (with-temp-file (expand-file-name "f.txt" dir)
+ (insert "changed\n"))
+ (let ((out (cj/gptel-git-diff--run dir)))
+ (should (string-match-p "^-original" out))
+ (should (string-match-p "^\\+changed" out))))))
+
+(ert-deftest test-gptel-tools-git-diff-run-narrow-to-file ()
+ "Normal: FILE argument narrows the diff."
+ (test-gptel-tools-git-diff--with-repo
+ (lambda (dir)
+ (with-temp-file (expand-file-name "f.txt" dir)
+ (insert "changed\n"))
+ (with-temp-file (expand-file-name "g.txt" dir)
+ (insert "second file\n"))
+ (let ((out (cj/gptel-git-diff--run dir nil nil "f.txt")))
+ (should (string-match-p "f.txt" out))
+ (should-not (string-match-p "g.txt" out))))))
+
+(ert-deftest test-gptel-tools-git-diff-run-error-on-bad-ref ()
+ "Error: git diff exits other than 0/1 are surfaced."
+ (test-gptel-tools-git-diff--with-repo
+ (lambda (dir)
+ (should-error (cj/gptel-git-diff--run dir "does-not-exist")))))
+
+(provide 'test-gptel-tools-git-diff)
+;;; test-gptel-tools-git-diff.el ends here