Konrad Mrożek
2023-07-13 22b826b7d0f82a3cfe99b43cd4819ad920fd8e41
Fix invalid line numbers in case of tail fail

`fail` means exception
1 files modified
27 ■■■■ changed files
clojure/src/test_runner.clj 27 ●●●● patch | view | raw | blame | history
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)))))