Watching stops at compile error or compile

[Kynde]

If I hit a compile error during file watching I can't seem to get the file watcher working again without a restart. Pressing Enter recompiles and runs all tests but still I can't get it to go back to watching files even then.

Experienced in auto mode:

• typo, :w
• compiler error
• fix typo, :w
• nothing happens

Expected:

• typo, :w
• compiler error
• fix typo, :w
• recompile and run all tests and continue in auto mode

Also experienced in auto mode:

• press enter
• runs all tests
• tweak a bit, :w
• nothing happens

Expected:

• press enter
• runs all tests
• tweak a bit, :w
• runs affected tests and continue is auto mode

Maybe I'm missing an option or my use case is off, but I thought I'd ask away.

[miikka]

You're right – I can reproduce this and it's a bug. The fix might be as simple as adding a try-catch block that eats the exceptions in the watchtower on-change handler:

bat-test/src/leiningen/bat_test.clj

Lines 52 to 54 in aaff8ff

	(watchtower.core/on-change (fn [~'_]
	(println)
	(metosin.bat-test.impl/run ~opts)))))

[Kynde]

Indeed. Works like a charm.

diff --git a/src/leiningen/bat_test.clj b/src/leiningen/bat_test.clj
index 00719ed..bd8afb1 100644
--- a/src/leiningen/bat_test.clj
+++ b/src/leiningen/bat_test.clj
@@ -51,7 +51,10 @@
               (watchtower.core/file-filter (watchtower.core/extensions :clj :cljc))
               (watchtower.core/on-change (fn [~'_]
                                            (println)
-                                           (metosin.bat-test.impl/run ~opts)))))
+                                           (try
+                                             (metosin.bat-test.impl/run ~opts)
+                                             (catch Exception e#
+                                               (println e#)))))))
         `(let [summary# (metosin.bat-test.impl/run ~opts)
                exit-code# (+ (:fail summary# 0) (:error summary# 0))]
            (if ~(= :leiningen (:eval-in project))

You drop it in or you want a PR out of it?

[miikka]

@Kynde - sorry for the slow reply, I was on vacation - please do send a PR!

[miikka]

The fix is released in [metosin/bat-test "0.4.3"] 🎉