Skip to content

Commit 5235ed0

Browse files
committed
detect failures in cljs test infrastructure
This makes a test fail, if require'ing the cljs testsuite fails on a combination of clojure/jdk >= 1.8/1.8 also revert a5553b5 because jenkins now has a recent jdk
1 parent a5553b5 commit 5235ed0

2 files changed

Lines changed: 13 additions & 7 deletions

File tree

src/test/clojure/clojure/data/xml/test_cljs.clj

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,15 @@
1616
(require 'clojure.data.xml.cljs-testsuite)
1717
(eval '(clojure.data.xml.cljs-testsuite/run-testsuite! "target/cljs-test-nashorn"))
1818
(catch Exception e
19-
(println "WARN: clojurescript test suite not available with Clojure"
20-
*clojure-version* (System/getProperty "java.runtime.name")
21-
(System/getProperty "java.vm.version") (System/getProperty "java.runtime.version")
22-
\newline e))))
19+
(if (or (neg? (compare ((juxt :major :minor) *clojure-version*)
20+
[1 8]))
21+
(neg? (compare (System/getProperty "java.runtime.version")
22+
"1.8")))
23+
(println "WARN: ignoring cljs testsuite error on clojure < 1.8 or jdk < 1.8"
24+
*clojure-version* (System/getProperty "java.runtime.name")
25+
(System/getProperty "java.vm.version") (System/getProperty "java.runtime.version")
26+
\newline (str e))
27+
(do (println "ERROR: cljs nashorn test suite should be able to run on clojure >= 1.8 and jdk >= 1.8"
28+
*clojure-version* (System/getProperty "java.runtime.name")
29+
(System/getProperty "java.vm.version") (System/getProperty "java.runtime.version"))
30+
(throw e))))))

src/test/resources/clojure/data/xml/cljs_testsuite.clj

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,7 @@
3434

3535
(defn run-testsuite! [dir]
3636
(System/setProperty "nashorn.persistent.code.cache" "target/nashorn_code_cache")
37-
(let [engine (repl-nh/create-engine
38-
;; wait for upgrade to more recent jdk8
39-
:code-cache false)]
37+
(let [engine (repl-nh/create-engine)]
4038
(println "INFO" "Running nashorn-repl with" (System/getProperty "nashorn.persistent.code.cache"))
4139
(compile-testsuite! dir)
4240
(.eval engine (io/reader (io/file dir "tests.reopt.js")))

0 commit comments

Comments
 (0)