blob: e6848014bbc591021c3c0b3a2c1e3ec7e1d6157c (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
|
;;; test-ai-vterm--reuse-existing-agent.el --- Tests for reuse-existing-agent action -*- lexical-binding: t; -*-
;;; Commentary:
;; The action looks for any window in the selected frame whose buffer
;; satisfies `cj/--ai-vterm-buffer-p'. When found, swaps that
;; window's buffer for the one being displayed and returns the
;; window. When not found, returns nil so the next action in the
;; chain runs.
;;
;; This is the action that keeps C-F9 (project-switch) from stealing
;; a non-agent window when the user is focused inside agent.
;;; Code:
(require 'ert)
(add-to-list 'load-path (expand-file-name "modules" user-emacs-directory))
(add-to-list 'load-path (expand-file-name "tests" user-emacs-directory))
(require 'ai-vterm)
(require 'testutil-vterm-buffers)
(ert-deftest test-ai-vterm--reuse-existing-agent-swaps-buffer-when-window-exists ()
"Normal: an agent window exists -> swap its buffer, return the window."
(cj/test--kill-agent-buffers)
(save-window-excursion
(delete-other-windows)
(let ((existing (get-buffer-create "agent [existing]"))
(new-buf (get-buffer-create "agent [new]"))
(split (split-window (selected-window) nil 'right)))
(unwind-protect
(progn
(set-window-buffer split existing)
(let ((result (cj/--ai-vterm-reuse-existing-agent new-buf nil)))
(should (eq result split))
(should (eq (window-buffer split) new-buf))))
(kill-buffer existing)
(kill-buffer new-buf)))))
(ert-deftest test-ai-vterm--reuse-existing-agent-returns-nil-when-no-agent-window ()
"Boundary: no agent window in frame -> nil (chain continues to next action)."
(cj/test--kill-agent-buffers)
(save-window-excursion
(delete-other-windows)
(let ((new-buf (get-buffer-create "agent [no-existing]")))
(unwind-protect
(should (null (cj/--ai-vterm-reuse-existing-agent new-buf nil)))
(kill-buffer new-buf)))))
(ert-deftest test-ai-vterm--reuse-existing-agent-leaves-non-agent-windows-alone ()
"Boundary: only non-agent windows in frame -> nil; other windows untouched."
(cj/test--kill-agent-buffers)
(save-window-excursion
(delete-other-windows)
(let ((code-buf (get-buffer-create "*test-code-buffer*"))
(new-agent (get-buffer-create "agent [new-here]"))
(other-win (split-window (selected-window) nil 'right)))
(unwind-protect
(progn
(set-window-buffer (selected-window) code-buf)
(set-window-buffer other-win code-buf)
(let ((result (cj/--ai-vterm-reuse-existing-agent
new-agent nil)))
(should (null result))
(should (eq (window-buffer (selected-window)) code-buf))
(should (eq (window-buffer other-win) code-buf))))
(kill-buffer code-buf)
(kill-buffer new-agent)))))
(ert-deftest test-ai-vterm--reuse-existing-agent-preserves-non-agent-window-when-swapping ()
"Normal: swap agent window only; the other window keeps its buffer.
This is the C-F9-from-agent regression: with agent at the bottom
and code on top, switching projects must replace the bottom window's
buffer, not the top window's."
(cj/test--kill-agent-buffers)
(save-window-excursion
(delete-other-windows)
(let* ((code-buf (get-buffer-create "*test-code-top*"))
(agent-a (get-buffer-create "agent [a]"))
(agent-b (get-buffer-create "agent [b]"))
(top-win (selected-window))
(bottom-win (split-window top-win nil 'below)))
(unwind-protect
(progn
(set-window-buffer top-win code-buf)
(set-window-buffer bottom-win agent-a)
;; Focus the agent window -- this is the regression scenario.
(select-window bottom-win)
(let ((result (cj/--ai-vterm-reuse-existing-agent
agent-b nil)))
(should (eq result bottom-win))
(should (eq (window-buffer bottom-win) agent-b))
(should (eq (window-buffer top-win) code-buf))))
(kill-buffer code-buf)
(kill-buffer agent-a)
(kill-buffer agent-b)))))
(provide 'test-ai-vterm--reuse-existing-agent)
;;; test-ai-vterm--reuse-existing-agent.el ends here
|