From 589629e36dddf2e0cd57f8c5fc6e3dd394000164 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Fri, 10 Oct 2025 11:42:50 +0000
Subject: [PATCH] fix(ctrader): Correct market open hours

The is_market_open method in the cTrader broker now accurately reflects forex market hours (Sunday 22:00 UTC to Friday 22:00 UTC). The previous implementation was a naive check that didn't account for the specific open and close times.
---
 core/brokers/ctrader_broker.py | 12 +++++--
 testing/test_ctrader_broker.py | 59 ++++++++++++++++++++++++++++++++++
 2 files changed, 68 insertions(+), 3 deletions(-)
 create mode 100644 testing/test_ctrader_broker.py

diff --git a/core/brokers/ctrader_broker.py b/core/brokers/ctrader_broker.py
index 0be477c..6de420b 100644
--- a/core/brokers/ctrader_broker.py
+++ b/core/brokers/ctrader_broker.py
@@ -399,9 +399,15 @@ class CTraderBroker(BaseBroker):
     
     def is_market_open(self) -> bool:
         """Check if forex market is open"""
-        now = datetime.now()
-        # Simplified: forex market closed on weekends
-        return now.weekday() < 5  # Monday=0, Sunday=6
+        now = datetime.utcnow()
+        # Forex market is open from Sunday 22:00 UTC to Friday 22:00 UTC
+        if now.weekday() == 5:  # Saturday
+            return False
+        if now.weekday() == 6 and now.hour < 22:  # Sunday before 22:00 UTC
+            return False
+        if now.weekday() == 4 and now.hour >= 22:  # Friday after 22:00 UTC
+            return False
+        return True
 
 # Convenience function
 def create_ctrader_broker(demo: bool = True) -> CTraderBroker:
diff --git a/testing/test_ctrader_broker.py b/testing/test_ctrader_broker.py
new file mode 100644
index 0000000..f693c9d
--- /dev/null
+++ b/testing/test_ctrader_broker.py
@@ -0,0 +1,59 @@
+# testing/test_ctrader_broker.py
+import unittest
+from unittest.mock import patch
+from datetime import datetime
+from core.brokers.ctrader_broker import CTraderBroker
+
+class TestCTraderBroker(unittest.TestCase):
+    """
+    Test cases for the cTrader broker implementation, focusing on market hours.
+    """
+
+    def setUp(self):
+        """Set up a CTraderBroker instance for testing."""
+        self.broker = CTraderBroker(demo=True)
+
+    @patch('core.brokers.ctrader_broker.datetime')
+    def test_is_market_open_weekday(self, mock_datetime):
+        """Test that the market is open on a standard weekday."""
+        # Wednesday, 12:00 UTC
+        mock_datetime.utcnow.return_value = datetime(2023, 1, 4, 12, 0, 0)
+        self.assertTrue(self.broker.is_market_open())
+
+    @patch('core.brokers.ctrader_broker.datetime')
+    def test_is_market_closed_saturday(self, mock_datetime):
+        """Test that the market is closed on Saturday."""
+        # Saturday, 12:00 UTC
+        mock_datetime.utcnow.return_value = datetime(2023, 1, 7, 12, 0, 0)
+        self.assertFalse(self.broker.is_market_open())
+
+    @patch('core.brokers.ctrader_broker.datetime')
+    def test_is_market_opens_sunday_evening(self, mock_datetime):
+        """Test that the market opens on Sunday evening."""
+        # Sunday, 22:01 UTC (market is open)
+        mock_datetime.utcnow.return_value = datetime(2023, 1, 8, 22, 1, 0)
+        self.assertTrue(self.broker.is_market_open())
+
+    @patch('core.brokers.ctrader_broker.datetime')
+    def test_is_market_closed_sunday_morning(self, mock_datetime):
+        """Test that the market is closed on Sunday morning."""
+        # Sunday, 10:00 UTC (market is closed)
+        mock_datetime.utcnow.return_value = datetime(2023, 1, 8, 10, 0, 0)
+        self.assertFalse(self.broker.is_market_open())
+
+    @patch('core.brokers.ctrader_broker.datetime')
+    def test_is_market_closes_friday_evening(self, mock_datetime):
+        """Test that the market closes on Friday evening."""
+        # Friday, 22:01 UTC (market is closed)
+        mock_datetime.utcnow.return_value = datetime(2023, 1, 6, 22, 1, 0)
+        self.assertFalse(self.broker.is_market_open())
+
+    @patch('core.brokers.ctrader_broker.datetime')
+    def test_is_market_open_friday_morning(self, mock_datetime):
+        """Test that the market is open on Friday morning."""
+        # Friday, 10:00 UTC (market is open)
+        mock_datetime.utcnow.return_value = datetime(2023, 1, 6, 10, 0, 0)
+        self.assertTrue(self.broker.is_market_open())
+
+if __name__ == '__main__':
+    unittest.main()