From bfe8d36e82660341b7d88df976c47a51a5cf6a2c Mon Sep 17 00:00:00 2001
From: 69049ed6x <69049ed6x@users.noreply.github.com>
Date: Sat, 7 Mar 2026 16:04:53 +0800
Subject: [PATCH] test: cover default factor rule rationale

---
 tests/test_factor_rules.py | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/tests/test_factor_rules.py b/tests/test_factor_rules.py
index 636344c4..526adcaf 100644
--- a/tests/test_factor_rules.py
+++ b/tests/test_factor_rules.py
@@ -224,6 +224,16 @@ class FactorRulesPathTests(unittest.TestCase):
         self.assertIn("- Weight: medium", summary)
         self.assertIn("- Thesis: Cheap relative to peers", summary)
 
+    def test_summarize_factor_rules_preserves_blank_rationale_line(self):
+        summary = summarize_factor_rules(
+            [{"name": "Carry", "signal": "neutral", "thesis": "Range-bound"}],
+            ticker="SPY",
+            trade_date="2026-03-07",
+        )
+
+        self.assertIn("- Rationale: ", summary)
+        self.assertIn("- Thesis: Range-bound", summary)
+
 
 if __name__ == "__main__":
     unittest.main()