Skip to content

Commit 8a77695

Browse files
committed
Only skip tests on windows, make note about them needing fixing
1 parent 5a96452 commit 8a77695

File tree

1 file changed

+11
-5
lines changed

1 file changed

+11
-5
lines changed

Eldev

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,17 +12,23 @@
1212
(eldev-add-extra-dependencies 'runtime '(:package logview :optional t))
1313
(eldev-add-extra-dependencies 'runtime '(:package clojure-ts-mode :optional t))
1414

15+
(defvar clojure-ts-failing-tests
16+
(when (member system-type '(ms-dos windows-nt cygwin))
17+
'("./test/cider-client-tests.el"
18+
"./test/cider-eval-tests.el"))
19+
"Tests that fail when running against clojure-ts-mode. These need to be fixed.
20+
See 'cider-expected-ns returns the namespace matching the given string path'
21+
and 'cider-provide-file can handle multibyte characters'
22+
test results from windows CI run.")
23+
1524
(defvar cider-test-type 'main)
1625
(setf eldev-standard-excludes `(:or ,eldev-standard-excludes
1726
;; Avoid including files in test "projects".
1827
(eldev-pcase-exhaustive cider-test-type
1928
(`main "./test/*/")
2029
(`integration '("./test/" "!./test/integration"))
21-
(`clojure-ts-mode '("./test/*/" "!./test/clojure-ts-mode"
22-
;; 1 test in each of the below files fails
23-
;; when running clojure-ts-mode tests on windows.
24-
"./test/cider-client-tests.el"
25-
"./test/cider-eval-tests.el"))
30+
(`clojure-ts-mode `("./test/*/" "!./test/clojure-ts-mode"
31+
,@clojure-ts-failing-tests))
2632
(`all '("./test/*/" "!./test/integration")))
2733
"test/integration/projects"
2834
;; This file is _supposed_ to be excluded

0 commit comments

Comments
 (0)