Skip to content

Commit e0ad599

Browse files
author
Bozhidar Batsov
committed
Add the Java version to the welcome banner & the connection info
1 parent 241c5d0 commit e0ad599

File tree

3 files changed

+20
-9
lines changed

3 files changed

+20
-9
lines changed

cider-interaction.el

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,6 +108,10 @@ which will use the default REPL connection."
108108
:group 'cider)
109109

110110
;;; Connection info
111+
(defun cider--java-version ()
112+
"Retrieve the underlying connection's Java version."
113+
(cider-eval-and-get-value "(System/getProperty \"java.version\")"))
114+
111115
(defun cider--clojure-version ()
112116
"Retrieve the underlying connection's Clojure version."
113117
(cider-eval-and-get-value "(clojure-version)"))
@@ -122,11 +126,12 @@ which will use the default REPL connection."
122126
Info contains project name, current REPL namespace, host:port
123127
endpoint and Clojure version."
124128
(with-current-buffer (get-buffer connection-buffer)
125-
(format "Active nREPL connection: %s:%s, %s:%s (Clojure %s, nREPL %s)"
129+
(format "Active nREPL connection: %s:%s, %s:%s (Java %s, Clojure %s, nREPL %s)"
126130
(or (nrepl--project-name nrepl-project-dir) "<no project>")
127131
nrepl-buffer-ns
128132
(car nrepl-endpoint)
129133
(cadr nrepl-endpoint)
134+
(cider--java-version)
130135
(cider--clojure-version)
131136
(cider--nrepl-version))))
132137

cider-repl.el

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -223,10 +223,11 @@ positions before and after executing BODY."
223223

224224
(defun cider-repl--banner ()
225225
"Generate the welcome REPL buffer banner."
226-
(format "; CIDER %s (Clojure %s, nREPL %s)"
226+
(format "; CIDER %s (Java %s, Clojure %s, nREPL %s)"
227227
(condition-case nil
228228
(pkg-info-version-info 'cider)
229229
(error cider-version))
230+
(cider--java-version)
230231
(cider--clojure-version)
231232
(cider--nrepl-version)))
232233

test/cider-tests.el

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -50,16 +50,18 @@
5050

5151
(ert-deftest test-cider-repl--banner ()
5252
(noflet ((pkg-info-version-info (library) "0.2.0")
53+
(cider--java-version () "1.7")
5354
(cider--clojure-version () "1.5.1")
5455
(cider--nrepl-version () "0.2.1"))
55-
(should (equal (cider-repl--banner) "; CIDER 0.2.0 (Clojure 1.5.1, nREPL 0.2.1)"))))
56+
(should (equal (cider-repl--banner) "; CIDER 0.2.0 (Java 1.7, Clojure 1.5.1, nREPL 0.2.1)"))))
5657

5758
(ert-deftest test-cider-repl--banner-version-fallback ()
5859
(noflet ((pkg-info-version-info (library) (error "No package version"))
60+
(cider--java-version () "1.7")
5961
(cider--clojure-version () "1.5.1")
6062
(cider--nrepl-version () "0.2.1"))
6163
(let ((cider-version "0.5.1"))
62-
(should (equal (cider-repl--banner) "; CIDER 0.5.1 (Clojure 1.5.1, nREPL 0.2.1)")))))
64+
(should (equal (cider-repl--banner) "; CIDER 0.5.1 (Java 1.7, Clojure 1.5.1, nREPL 0.2.1)")))))
6365

6466
(ert-deftest test-cider-var-info ()
6567
(noflet ((nrepl-send-request-sync (list)
@@ -127,7 +129,8 @@
127129
(let ((nrepl-connection-list
128130
(list (buffer-name a) (buffer-name b) (buffer-name c)))
129131
(nrepl-connection-buffer nil))
130-
(noflet ((cider--clojure-version () "")
132+
(noflet ((cider--java-version () "")
133+
(cider--clojure-version () "")
131134
(cider--nrepl-version () ""))
132135
(should (equal (buffer-name a) (nrepl-current-connection-buffer)))
133136
(cider-rotate-connection)
@@ -139,22 +142,24 @@
139142

140143
(ert-deftest test-cider--current-connection-info ()
141144
(with-temp-buffer
142-
(noflet ((cider--clojure-version () "1.5.1")
145+
(noflet ((cider--java-version () "1.7")
146+
(cider--clojure-version () "1.5.1")
143147
(cider--nrepl-version () "0.2.1"))
144148
(setq-local nrepl-endpoint '("localhost" 4005))
145149
(setq-local nrepl-project-dir "proj")
146150
(setq-local nrepl-buffer-ns "somens")
147151
(should (string= (cider--connection-info (buffer-name (current-buffer)))
148-
"Active nREPL connection: proj:somens, localhost:4005 (Clojure 1.5.1, nREPL 0.2.1)")))))
152+
"Active nREPL connection: proj:somens, localhost:4005 (Java 1.7, Clojure 1.5.1, nREPL 0.2.1)")))))
149153

150154
(ert-deftest test-cider-current-connection-info-no-project ()
151155
(with-temp-buffer
152-
(noflet ((cider--clojure-version () "1.5.1")
156+
(noflet ((cider--java-version () "1.7")
157+
(cider--clojure-version () "1.5.1")
153158
(cider--nrepl-version () "0.2.1"))
154159
(setq-local nrepl-endpoint '("localhost" 4005))
155160
(setq-local nrepl-buffer-ns "somens")
156161
(should (string= (cider--connection-info (buffer-name (current-buffer)))
157-
"Active nREPL connection: <no project>:somens, localhost:4005 (Clojure 1.5.1, nREPL 0.2.1)")))))
162+
"Active nREPL connection: <no project>:somens, localhost:4005 (Java 1.7, Clojure 1.5.1, nREPL 0.2.1)")))))
158163

159164
(ert-deftest test-nrepl-close ()
160165
(let ((connections (nrepl-connection-buffers)))

0 commit comments

Comments
 (0)