From 88cddc0fe56a7d23570ce108bcf3f02e446edd59 Mon Sep 17 00:00:00 2001 From: Konrad Mrożek <kmrozek@comscore.com> Date: Thu, 31 Aug 2023 10:20:02 +0000 Subject: [PATCH] Treat malli errors as exceptions --- clojure/src/test_runner.clj | 44 +++++++++++++++++++++++++------------------- 1 files changed, 25 insertions(+), 19 deletions(-) diff --git a/clojure/src/test_runner.clj b/clojure/src/test_runner.clj index 209c826..53ee877 100644 --- a/clojure/src/test_runner.clj +++ b/clojure/src/test_runner.clj @@ -1,11 +1,15 @@ -(binding [*compile-files* true] - (require 'clojure.test)) +(ns test-runner) -(ns test-runner - (:require [clojure.test :as t])) +(def class-cache-dir ".cache/classes") +(.mkdirs (java.io.File. class-cache-dir)) -(when-not (.exists (java.io.File. "classes")) - (.. (java.io.File. "classes") mkdir)) +(defmacro with-class-cache [& body] + `(binding [*compile-path* class-cache-dir + *compile-files* true] + ~@body)) + +(with-class-cache + (require '[clojure.test :as t])) (defmulti vim-report :type) @@ -17,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)) @@ -40,6 +44,7 @@ (defmethod vim-report :error [m] + (t/inc-report-counter :error) (when-let [source-file (some-> t/*testing-vars* first meta @@ -71,7 +76,7 @@ first)) (defn -main [& {:strs [-test-file -test-line] :or {-test-file "test"}}] - (binding [*compile-files* true] + (with-class-cache (compile 'test-runner) (println "Detecting test files in" -test-file) (let [test-files (->> -test-file @@ -86,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)) @@ -97,8 +103,9 @@ (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))])] + + (let [results (t/run-test-var (find-closest-test (.. (java.io.File. -test-file) getAbsolutePath) + (parse-long -test-line)))] (+ (:fail results 0) (:error results 0))) (reduce (fn [total-fails n] @@ -110,7 +117,6 @@ test-namespaces))) 1 0))))))) - -- Gitblit v1.9.3