From 22b826b7d0f82a3cfe99b43cd4819ad920fd8e41 Mon Sep 17 00:00:00 2001
From: Konrad Mrożek <konrad.mrozek@shareablee.com>
Date: Thu, 13 Jul 2023 13:13:05 +0000
Subject: [PATCH] Fix invalid line numbers in case of tail fail

---
 clojure/src/test_runner.clj |   27 ++++++++++++++++++++-------
 1 files changed, 20 insertions(+), 7 deletions(-)

diff --git a/clojure/src/test_runner.clj b/clojure/src/test_runner.clj
index ed5ab46..82ba123 100644
--- a/clojure/src/test_runner.clj
+++ b/clojure/src/test_runner.clj
@@ -20,18 +20,31 @@
       (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))
+    (let [fname (-> source-file (java.io.File.) (.getName))]
+      (->> m
+           :actual
+           Throwable->map
+           :trace
+           (some (fn [[_ _ e-file e-line]]
+                   (when (= e-file fname)
+                     e-line)))))
+    (:line m)))
+
 (defmethod emacs-report :error
   [m]
   (when-let [source-file (some-> t/*testing-vars*
                                  first
                                  meta
                                  :file)]
-    (println (str "ERROR:" source-file ":" (:line m) ":" (t/testing-vars-str m) ":" (t/testing-contexts-str) ":" (:message m "FAIL")))
-    (println (str "ERROR-CONTINUE:EXPECTED:" (pr-str (:expected m))))
-    (println (str "ERROR-CONTINUE:ACTUAL:"
-                  (if (instance? Throwable (:actual m))
-                    (ex-message (:actual m))
-                    (pr-str (:actual m)))))))
+    (let [line (find-line-number source-file m)]
+      (println (str "ERROR:" source-file ":" line ":" (t/testing-vars-str m) ":" (t/testing-contexts-str) ":" (:message m "FAIL")))
+      (println (str "ERROR-CONTINUE:EXPECTED:" (pr-str (:expected m))))
+      (println (str "ERROR-CONTINUE:ACTUAL:"
+                    (if (instance? Throwable (:actual m))
+                      (ex-message (:actual m))
+                      (pr-str (:actual m))))))))
 
 (defmethod emacs-report :default
   [_])
@@ -62,7 +75,7 @@
         (println "Running tests on:" (map ns-name test-namespaces))
         (System/exit (if (pos? (reduce (fn [total-fails n]
                                          (with-redefs [t/report emacs-report]
-                                           (let [results  (t/run-tests n)]
+                                           (let [results (t/run-tests n)]
                                              (+ total-fails
                                                 (:fail results 0)
                                                 (:error results 0)))))

--
Gitblit v1.9.3