Restart monitor after signal watchlist refresh

Author: WW-shan

scripts/background_manager.sh

@@ -269,7 +269,12 @@ case "$COMMAND" in
       fi
 
       if [[ "$ENABLE_EXTERNAL_SIGNALS" == "1" && "$now" -ge "$next_external" ]]; then
+        before_sig="$(file_sig "$WATCHLIST")"
         run_logged external-signals scripts/refresh_external_signals.sh
+        after_sig="$(file_sig "$WATCHLIST")"
+        if [[ "$before_sig" != "$after_sig" ]]; then
+          restart_monitor
+        fi
         next_external=$((now + EXTERNAL_SIGNALS_INTERVAL))
       fi
 

tests/test_background_manager_script.py

@@ -0,0 +1,20 @@
+import unittest
+from pathlib import Path
+
+
+class BackgroundManagerScriptTests(unittest.TestCase):
+    def test_external_signal_refresh_restarts_monitor_when_watchlist_changes(self):
+        root = Path(__file__).resolve().parents[1]
+        text = (root / "scripts" / "background_manager.sh").read_text()
+        block = text.split('if [[ "$ENABLE_EXTERNAL_SIGNALS" == "1"')[1].split(
+            'if [[ "$ENABLE_DISCOVERY_REFRESH" == "1"'
+        )[0]
+
+        self.assertIn('before_sig="$(file_sig "$WATCHLIST")"', block)
+        self.assertIn('after_sig="$(file_sig "$WATCHLIST")"', block)
+        self.assertIn('if [[ "$before_sig" != "$after_sig" ]]', block)
+        self.assertIn("restart_monitor", block)
+
+
+if __name__ == "__main__":
+    unittest.main()