From 70423acb0fa4343100b879664757ea12750a875c Mon Sep 17 00:00:00 2001 From: "zihao.jiang" Date: Tue, 6 Jan 2026 14:13:29 +0800 Subject: [PATCH 1/3] add back get_open_interest_history() function --- okx/TradingData.py | 2 +- test/test_trading_data.py | 4 ++-- test/unit/okx/test_trading_data.py | 14 +++++++------- 3 files changed, 10 insertions(+), 10 deletions(-) diff --git a/okx/TradingData.py b/okx/TradingData.py index 0a0d3e4..0c90070 100644 --- a/okx/TradingData.py +++ b/okx/TradingData.py @@ -47,7 +47,7 @@ def get_taker_block_volume(self, ccy, period=''): params = {'ccy': ccy, 'period': period} return self._request_with_params(GET, TAKER_FLOW, params) - def get_contracts_open_interest_history(self, instId, period=None, begin=None, end=None, limit=None): + def get_open_interest_history(self, instId, period=None, begin=None, end=None, limit=None): """ Get contract open interest history Retrieve the contract open interest statistics of futures and perp. diff --git a/test/test_trading_data.py b/test/test_trading_data.py index ecb6c49..b535b78 100644 --- a/test/test_trading_data.py +++ b/test/test_trading_data.py @@ -31,10 +31,10 @@ def test_open_interest_volume_strike(self): """ def test_get_contracts_open_interest_history(self): - print(self.TradingDataAPI.get_contracts_open_interest_history(instId='BTC-USDT-SWAP')) + print(self.TradingDataAPI.get_open_interest_history(instId='BTC-USDT-SWAP')) def test_get_contracts_open_interest_history_with_params(self): - print(self.TradingDataAPI.get_contracts_open_interest_history( + print(self.TradingDataAPI.get_open_interest_history( instId='BTC-USDT-SWAP', period='1H', limit='50' diff --git a/test/unit/okx/test_trading_data.py b/test/unit/okx/test_trading_data.py index c710a76..e8dc292 100644 --- a/test/unit/okx/test_trading_data.py +++ b/test/unit/okx/test_trading_data.py @@ -30,7 +30,7 @@ def test_get_contracts_open_interest_history_with_required_params(self, mock_req mock_request.return_value = mock_response # Act - result = self.trading_data_api.get_contracts_open_interest_history( + result = self.trading_data_api.get_open_interest_history( instId='BTC-USDT-SWAP' ) @@ -55,7 +55,7 @@ def test_get_contracts_open_interest_history_with_all_params(self, mock_request) mock_request.return_value = mock_response # Act - result = self.trading_data_api.get_contracts_open_interest_history( + result = self.trading_data_api.get_open_interest_history( instId='BTC-USDT-SWAP', period='1H', begin='1609459200000', @@ -82,7 +82,7 @@ def test_get_contracts_open_interest_history_with_period(self, mock_request): mock_request.return_value = mock_response # Act - result = self.trading_data_api.get_contracts_open_interest_history( + result = self.trading_data_api.get_open_interest_history( instId='ETH-USDT-SWAP', period='5m' ) @@ -104,7 +104,7 @@ def test_get_contracts_open_interest_history_different_periods(self, mock_reques for period in periods: mock_request.reset_mock() - result = self.trading_data_api.get_contracts_open_interest_history( + result = self.trading_data_api.get_open_interest_history( instId='BTC-USDT-SWAP', period=period ) @@ -122,7 +122,7 @@ def test_get_contracts_open_interest_history_different_inst_ids(self, mock_reque for inst_id in inst_ids: mock_request.reset_mock() - result = self.trading_data_api.get_contracts_open_interest_history( + result = self.trading_data_api.get_open_interest_history( instId=inst_id ) @@ -137,7 +137,7 @@ def test_get_contracts_open_interest_history_with_pagination(self, mock_request) mock_request.return_value = mock_response # Act - result = self.trading_data_api.get_contracts_open_interest_history( + result = self.trading_data_api.get_open_interest_history( instId='BTC-USDT-SWAP', begin='1609459200000', end='1609545600000', @@ -163,7 +163,7 @@ def test_get_contracts_open_interest_history_utc_periods(self, mock_request): for period in utc_periods: mock_request.reset_mock() - result = self.trading_data_api.get_contracts_open_interest_history( + result = self.trading_data_api.get_open_interest_history( instId='BTC-USDT-SWAP', period=period ) From 96c1fdb4ea380dbf803c829f1792284050be8f79 Mon Sep 17 00:00:00 2001 From: "zihao.jiang" Date: Tue, 6 Jan 2026 14:19:22 +0800 Subject: [PATCH 2/3] add back get_open_interest_history() function --- test/test_trading_data.py | 4 ++-- test/unit/okx/test_trading_data.py | 30 +++++++++++++++--------------- 2 files changed, 17 insertions(+), 17 deletions(-) diff --git a/test/test_trading_data.py b/test/test_trading_data.py index b535b78..0b9e0b8 100644 --- a/test/test_trading_data.py +++ b/test/test_trading_data.py @@ -30,10 +30,10 @@ def test_open_interest_volume_strike(self): """ - def test_get_contracts_open_interest_history(self): + def test_get_open_interest_history(self): print(self.TradingDataAPI.get_open_interest_history(instId='BTC-USDT-SWAP')) - def test_get_contracts_open_interest_history_with_params(self): + def test_get_open_interest_history_with_params(self): print(self.TradingDataAPI.get_open_interest_history( instId='BTC-USDT-SWAP', period='1H', diff --git a/test/unit/okx/test_trading_data.py b/test/unit/okx/test_trading_data.py index e8dc292..db81f8c 100644 --- a/test/unit/okx/test_trading_data.py +++ b/test/unit/okx/test_trading_data.py @@ -10,15 +10,15 @@ class TestTradingDataAPIContractsOpenInterestHistory(unittest.TestCase): - """Unit tests for the get_contracts_open_interest_history method""" + """Unit tests for the get_open_interest_history method""" def setUp(self): """Set up test fixtures""" self.trading_data_api = TradingDataAPI(flag='0') @patch.object(TradingDataAPI, '_request_with_params') - def test_get_contracts_open_interest_history_with_required_params(self, mock_request): - """Test get_contracts_open_interest_history with required parameters only""" + def test_get_open_interest_history_with_required_params(self, mock_request): + """Test ge_open_interest_history with required parameters only""" # Arrange mock_response = { 'code': '0', @@ -42,8 +42,8 @@ def test_get_contracts_open_interest_history_with_required_params(self, mock_req self.assertEqual(result, mock_response) @patch.object(TradingDataAPI, '_request_with_params') - def test_get_contracts_open_interest_history_with_all_params(self, mock_request): - """Test get_contracts_open_interest_history with all parameters provided""" + def test_get_open_interest_history_with_all_params(self, mock_request): + """Test get_open_interest_history with all parameters provided""" # Arrange mock_response = { 'code': '0', @@ -75,8 +75,8 @@ def test_get_contracts_open_interest_history_with_all_params(self, mock_request) self.assertEqual(result, mock_response) @patch.object(TradingDataAPI, '_request_with_params') - def test_get_contracts_open_interest_history_with_period(self, mock_request): - """Test get_contracts_open_interest_history with period parameter""" + def test_get_open_interest_history_with_period(self, mock_request): + """Test get_open_interest_history with period parameter""" # Arrange mock_response = {'code': '0', 'msg': '', 'data': []} mock_request.return_value = mock_response @@ -95,8 +95,8 @@ def test_get_contracts_open_interest_history_with_period(self, mock_request): mock_request.assert_called_once_with(c.GET, c.CONTRACTS_OPEN_INTEREST_HISTORY, expected_params) @patch.object(TradingDataAPI, '_request_with_params') - def test_get_contracts_open_interest_history_different_periods(self, mock_request): - """Test get_contracts_open_interest_history with different period values""" + def test_get_open_interest_history_different_periods(self, mock_request): + """Test get_open_interest_history with different period values""" mock_response = {'code': '0', 'msg': '', 'data': []} mock_request.return_value = mock_response @@ -113,8 +113,8 @@ def test_get_contracts_open_interest_history_different_periods(self, mock_reques self.assertEqual(call_args['period'], period) @patch.object(TradingDataAPI, '_request_with_params') - def test_get_contracts_open_interest_history_different_inst_ids(self, mock_request): - """Test get_contracts_open_interest_history with different instrument IDs""" + def test_get_open_interest_history_different_inst_ids(self, mock_request): + """Test get_open_interest_history with different instrument IDs""" mock_response = {'code': '0', 'msg': '', 'data': []} mock_request.return_value = mock_response @@ -130,8 +130,8 @@ def test_get_contracts_open_interest_history_different_inst_ids(self, mock_reque self.assertEqual(call_args['instId'], inst_id) @patch.object(TradingDataAPI, '_request_with_params') - def test_get_contracts_open_interest_history_with_pagination(self, mock_request): - """Test get_contracts_open_interest_history with pagination parameters""" + def test_get_open_interest_history_with_pagination(self, mock_request): + """Test get_open_interest_history with pagination parameters""" # Arrange mock_response = {'code': '0', 'msg': '', 'data': []} mock_request.return_value = mock_response @@ -154,8 +154,8 @@ def test_get_contracts_open_interest_history_with_pagination(self, mock_request) mock_request.assert_called_once_with(c.GET, c.CONTRACTS_OPEN_INTEREST_HISTORY, expected_params) @patch.object(TradingDataAPI, '_request_with_params') - def test_get_contracts_open_interest_history_utc_periods(self, mock_request): - """Test get_contracts_open_interest_history with UTC+0 period values""" + def test_get_open_interest_history_utc_periods(self, mock_request): + """Test get_open_interest_history with UTC+0 period values""" mock_response = {'code': '0', 'msg': '', 'data': []} mock_request.return_value = mock_response From 4343adc08fc2621c6337a107952af1d32c6a592a Mon Sep 17 00:00:00 2001 From: "zihao.jiang" Date: Tue, 6 Jan 2026 14:24:32 +0800 Subject: [PATCH 3/3] add back get_open_interest_history() function --- test/test_trading_data.py | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/test/test_trading_data.py b/test/test_trading_data.py index 0b9e0b8..99eaabd 100644 --- a/test/test_trading_data.py +++ b/test/test_trading_data.py @@ -30,15 +30,15 @@ def test_open_interest_volume_strike(self): """ - def test_get_open_interest_history(self): - print(self.TradingDataAPI.get_open_interest_history(instId='BTC-USDT-SWAP')) - - def test_get_open_interest_history_with_params(self): - print(self.TradingDataAPI.get_open_interest_history( - instId='BTC-USDT-SWAP', - period='1H', - limit='50' - )) + # def test_get_open_interest_history(self): + # print(self.TradingDataAPI.get_open_interest_history(instId='BTC-USDT-SWAP')) + # + # def test_get_open_interest_history_with_params(self): + # print(self.TradingDataAPI.get_open_interest_history( + # instId='BTC-USDT-SWAP', + # period='1H', + # limit='50' + # )) if __name__ == "__main__": unittest.main()