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..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_contracts_open_interest_history(self): - print(self.TradingDataAPI.get_contracts_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( - 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() diff --git a/test/unit/okx/test_trading_data.py b/test/unit/okx/test_trading_data.py index c710a76..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', @@ -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' ) @@ -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', @@ -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', @@ -75,14 +75,14 @@ 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 # 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' ) @@ -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 @@ -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 ) @@ -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 @@ -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 ) @@ -130,14 +130,14 @@ 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 # 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', @@ -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 @@ -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 )