From d937545d69736a906fdb549b75bdf2b8e005e287 Mon Sep 17 00:00:00 2001 From: Konrad Mrożek <kmrozek@comscore.com> Date: Thu, 31 Aug 2023 10:20:20 +0000 Subject: [PATCH] Handle exit code correctly --- clojure/src/test_runner.clj | 33 ++++++++++++++++++++------------- 1 files changed, 20 insertions(+), 13 deletions(-) diff --git a/clojure/src/test_runner.clj b/clojure/src/test_runner.clj index 57864bc..af558db 100644 --- a/clojure/src/test_runner.clj +++ b/clojure/src/test_runner.clj @@ -21,14 +21,14 @@ (defmethod vim-report :fail [m] - (t/with-test-out - (when-let [source-file (some-> t/*testing-vars* - first - meta - :file)] - (println (str "FAIL:" source-file ":" (:line m) ":" (t/testing-vars-str m) ":" (t/testing-contexts-str) ":" (:message m "FAIL"))) - (println (str "FAIL-CONTINUE:EXPECTED:" (pr-str (:expected m)))) - (println (str "FAIL-CONTINUE:ACTUAL:" (pr-str (:actual m))))))) + (t/inc-report-counter :fail) + (when-let [source-file (some-> t/*testing-vars* + first + meta + :file)] + (println (str "FAIL:" source-file ":" (:line m) ":" (t/testing-vars-str m) ":" (t/testing-contexts-str) ":" (:message m "FAIL"))) + (println (str "FAIL-CONTINUE:EXPECTED:" (pr-str (:expected m)))) + (println (str "FAIL-CONTINUE:ACTUAL:" (pr-str (:actual m)))))) (defn- find-line-number [source-file m] (if (instance? Throwable (:actual m)) @@ -44,6 +44,7 @@ (defmethod vim-report :error [m] + (t/inc-report-counter :error) (when-let [source-file (some-> t/*testing-vars* first meta @@ -90,7 +91,8 @@ (when (find-ns 'malli.core) (println "Malli detected. Instrument functions...") (require 'malli.dev) - ((find-var 'malli.dev/start!))) + (require 'malli.dev.pretty) + ((find-var 'malli.dev/start!) {:report ((find-var 'malli.dev.pretty/thrower))})) (let [test-namespaces (->> (all-ns) (mapcat ns-publics) (map (comp meta second)) @@ -101,10 +103,15 @@ (with-redefs [t/report vim-report] (System/exit (if (pos? (if (and -test-file -test-line) - (let [results (t/test-vars [(find-closest-test (.. (java.io.File. -test-file) getAbsolutePath) - (parse-long -test-line))])] - (+ (:fail results 0) - (:error results 0))) + + (if-let [test-var (find-closest-test (.. (java.io.File. -test-file) getAbsolutePath) + (parse-long -test-line))] + (->> (t/run-test-var test-var) + ((juxt :fail :error)) + (apply +)) + (do + (println "No test found") + 0)) (reduce (fn [total-fails n] (let [results (t/run-tests n)] (+ total-fails -- Gitblit v1.9.3