Skip to content

Commit 8019863

Browse files
committed
Print the dart sdk version on
1 parent f7ef9c2 commit 8019863

File tree

4 files changed

+27
-12
lines changed

4 files changed

+27
-12
lines changed

lsp-dart-utils.el

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -156,10 +156,14 @@ FLUTTER_ROOT environment variable."
156156

157157
;; Version
158158

159-
(defun lsp-dart--get-dart-version ()
159+
(defun lsp-dart-get-full-dart-version ()
160160
"Retrieve the dart version from shell command."
161161
(->> (concat (lsp-dart-dart-command) " --version")
162-
shell-command-to-string
162+
shell-command-to-string))
163+
164+
(defun lsp-dart--get-dart-version ()
165+
"Retrieve the dart version from shell command."
166+
(->> (lsp-dart-get-full-dart-version)
163167
split-string
164168
(nth 3)))
165169

lsp-dart.el

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -126,12 +126,15 @@ The returned string includes the version from main file header,
126126
If the version number could not be determined, signal an error."
127127
(interactive)
128128
(if (require 'pkg-info nil t)
129-
(let ((version (pkg-info-version-info 'lsp-dart)))
130-
(lsp-dart-log
131-
"%s at %s @ Emacs %s"
132-
version
133-
(format-time-string "%Y.%m.%d" (current-time))
134-
emacs-version))
129+
(let* ((version (pkg-info-version-info 'lsp-dart))
130+
(lsp-dart-string (format "%s at %s @ Emacs %s"
131+
version
132+
(format-time-string "%Y.%m.%d" (current-time))
133+
emacs-version))
134+
(dart-sdk-string (concat (propertize "[Dart SDK] "
135+
'face 'font-lock-function-name-face)
136+
(lsp-dart-get-full-dart-version))))
137+
(lsp-dart-log "%s\n%s" lsp-dart-string dart-sdk-string))
135138
(error "Cannot determine version without package 'pkg-info'")))
136139

137140

test/lsp-dart-test.el

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -73,9 +73,11 @@
7373

7474
(ert-deftest lsp-dart-version--test ()
7575
(let ((pkg-version (lsp-dart-test-package-version "lsp-dart.el")))
76-
(should (equal (lsp-dart-version) (format "[LSP Dart] %s at %s @ Emacs %s"
77-
pkg-version
78-
(format-time-string "%Y.%m.%d" (current-time))
79-
emacs-version)))))
76+
(with-mock
77+
(stub lsp-dart-get-full-dart-version => "2.8.2")
78+
(should (equal (lsp-dart-version) (format "[LSP Dart] %s at %s @ Emacs %s\n[Dart SDK] 2.8.2"
79+
pkg-version
80+
(format-time-string "%Y.%m.%d" (current-time))
81+
emacs-version))))))
8082

8183
;;; lsp-dart-test.el ends here

test/lsp-dart-utils-test.el

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -156,6 +156,12 @@
156156
(mock (lsp-workspace-status nil "workspace"))
157157
(lsp-dart-workspace-status nil "workspace")))
158158

159+
(ert-deftest lsp-dart--get-full-dart-version--test ()
160+
(with-mock
161+
(mock (lsp-dart-dart-command) => "dart")
162+
(mock (shell-command-to-string "dart --version") => "Dart VM version: 2.9.0-4.0.dev")
163+
(should (equal (lsp-dart-get-full-dart-version) "Dart VM version: 2.9.0-4.0.dev"))))
164+
159165
(ert-deftest lsp-dart--get-dart-version--test ()
160166
(with-mock
161167
(mock (lsp-dart-dart-command) => "dart")

0 commit comments

Comments
 (0)