Fix the --no-notifications CLI flag for the notifier plugin

There was a logic error where it would trigger the same behavior as if no
suitable notifier command was found, i.e. fall back to Java desktop tray
notification handling.

Author: plexus

CHANGELOG.md

@@ -1,18 +1,18 @@
 # Unreleased
 
-- Add specs for kaocha.plugin/profiling data (#302).
-
 ## Added
 
+- Add specs for kaocha.plugin/profiling data (#302).
+
 ## Fixed
 
 - Fix issue where diffing `(is (= ...))` assertions sometimes fails
   when comparing records.
+- Fix the `--no-notifications` flag in the Notifier plugin
 
 ## Changed
 
 - Some error codes were duplicated. This is a **breaking change** if you rely on error codes. 
-
     - When a test suite configuration value is not a collection or symbol, the
 	error code is now 250 instead of 252. The error code 252 is still used
 	when the configuration file fails to load.

src/kaocha/plugin/notifier.clj

@@ -157,7 +157,8 @@
                (::notifications? config true)))))
 
   (post-run [result]
-    (if-let [command (and (::notifications? result) (::command result))]
-      (run-command command result)
-      (send-tray-notification result))
+    (when (::notifications? result)
+      (if-let [command (::command result)]
+        (run-command command result)
+        (send-tray-notification result)))
     result))