Skip to content

Commit 812f652

Browse files
authored
chore: update ids (#1692)
* chore: update ids * ignore file * add id tests to python * fix tests
1 parent b697c97 commit 812f652

4 files changed

Lines changed: 27 additions & 24 deletions

File tree

.github/workflows/python-app.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,8 @@ jobs:
6363
pip install pytest pytest-cov pyright tox
6464
if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
6565
if [ -f test-requirements.txt ]; then pip install -r test-requirements.txt; fi
66+
- name: Id Tests
67+
run: pytest tests/test_ids.py
6668
- name: Type check with pyright (Python 3.12 only)
6769
if: matrix.python-version == '3.12'
6870
run: pyright

binance/base_client.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -152,8 +152,8 @@ class BaseClient:
152152
MINING_TO_FIAT = "MINING_C2C"
153153

154154
## order ids
155-
SPOT_ORDER_PREFIX = "x-HNA2TXFJ"
156-
CONTRACT_ORDER_PREFIX = "x-Cb7ytekJ"
155+
SPOT_ORDER_PREFIX = "x-TKT5PX2F"
156+
CONTRACT_ORDER_PREFIX = "x-cvBPrNm9"
157157

158158
def __init__(
159159
self,

pyproject.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ lint.ignore = [
1414
"SIM102","SIM117","SIM118","SIM401",
1515
"TRY002","TRY004","TRY201",
1616
]
17+
exclude = ["generate_llms_txt.py"]
1718

1819
[tool.pytest.ini_options]
1920
timeout = 90

tests/test_ids.py

Lines changed: 22 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ def test_spot_id():
1717
assert url_dict["side"] == "BUY"
1818
assert url_dict["type"] == "MARKET"
1919
assert url_dict["quantity"] == "0.1"
20-
assert url_dict["newClientOrderId"].startswith("x-HNA2TXFJ")
20+
assert url_dict["newClientOrderId"].startswith("x-TKT5PX2F")
2121

2222

2323
def test_spot_limit_id():
@@ -27,7 +27,7 @@ def test_spot_limit_id():
2727
symbol="LTCUSDT", side="BUY", type="MARKET", quantity=0.1
2828
)
2929
url_dict = dict(pair.split("=") for pair in m.last_request.text.split("&"))
30-
assert url_dict["newClientOrderId"].startswith("x-HNA2TXFJ")
30+
assert url_dict["newClientOrderId"].startswith("x-TKT5PX2F")
3131

3232

3333
def test_spot_market_id():
@@ -37,7 +37,7 @@ def test_spot_market_id():
3737
symbol="LTCUSDT", side="BUY", type="MARKET", quantity=0.1
3838
)
3939
url_dict = dict(pair.split("=") for pair in m.last_request.text.split("&"))
40-
assert url_dict["newClientOrderId"].startswith("x-HNA2TXFJ")
40+
assert url_dict["newClientOrderId"].startswith("x-TKT5PX2F")
4141

4242

4343
def test_spot_cancel_replace_id():
@@ -55,7 +55,7 @@ def test_spot_cancel_replace_id():
5555
quantity=0.1,
5656
)
5757
url_dict = dict(pair.split("=") for pair in m.last_request.text.split("&"))
58-
assert url_dict["newClientOrderId"].startswith("x-HNA2TXFJ")
58+
assert url_dict["newClientOrderId"].startswith("x-TKT5PX2F")
5959

6060

6161
def test_spot_oco_order_id():
@@ -65,7 +65,7 @@ def test_spot_oco_order_id():
6565
symbol="LTCUSDT", side="BUY", aboveType="MARKET", quantity=0.1
6666
)
6767
url_dict = dict(pair.split("=") for pair in m.last_request.text.split("&"))
68-
assert url_dict["listClientOrderId"].startswith("x-HNA2TXFJ")
68+
assert url_dict["listClientOrderId"].startswith("x-TKT5PX2F")
6969

7070

7171
def test_swap_id():
@@ -80,7 +80,7 @@ def test_swap_id():
8080
assert url_dict["side"] == "BUY"
8181
assert url_dict["type"] == "MARKET"
8282
assert url_dict["quantity"] == "0.1"
83-
assert url_dict["newClientOrderId"].startswith("x-Cb7ytekJ")
83+
assert url_dict["newClientOrderId"].startswith("x-cvBPrNm9")
8484

8585

8686
def test_swap_batch_id():
@@ -90,7 +90,7 @@ def test_swap_batch_id():
9090
orders = [order, order]
9191
client.futures_place_batch_order(batchOrders=orders)
9292
text = m.last_request.text
93-
assert "x-Cb7ytekJ" in text
93+
assert "x-cvBPrNm9" in text
9494

9595

9696
def test_coin_id():
@@ -105,7 +105,7 @@ def test_coin_id():
105105
assert url_dict["side"] == "BUY"
106106
assert url_dict["type"] == "MARKET"
107107
assert url_dict["quantity"] == "0.1"
108-
assert url_dict["newClientOrderId"].startswith("x-Cb7ytekJ")
108+
assert url_dict["newClientOrderId"].startswith("x-cvBPrNm9")
109109

110110

111111
def test_coin_batch_id():
@@ -120,7 +120,7 @@ def test_coin_batch_id():
120120
orders = [order, order]
121121
client.futures_coin_place_batch_order(batchOrders=orders)
122122
text = m.last_request.text
123-
assert "x-Cb7ytekJ" in text
123+
assert "x-cvBPrNm9" in text
124124

125125

126126
def test_papi_um_id():
@@ -135,7 +135,7 @@ def test_papi_um_id():
135135
assert url_dict["side"] == "BUY"
136136
assert url_dict["type"] == "MARKET"
137137
assert url_dict["quantity"] == "0.1"
138-
assert url_dict["newClientOrderId"].startswith("x-Cb7ytekJ")
138+
assert url_dict["newClientOrderId"].startswith("x-cvBPrNm9")
139139

140140

141141
def test_papi_cm_id():
@@ -150,7 +150,7 @@ def test_papi_cm_id():
150150
assert url_dict["side"] == "BUY"
151151
assert url_dict["type"] == "MARKET"
152152
assert url_dict["quantity"] == "0.1"
153-
assert url_dict["newClientOrderId"].startswith("x-Cb7ytekJ")
153+
assert url_dict["newClientOrderId"].startswith("x-cvBPrNm9")
154154

155155

156156
@pytest.mark.asyncio()
@@ -162,7 +162,7 @@ async def test_spot_id_async():
162162

163163
def handler(url, **kwargs):
164164
client_order_id = kwargs["data"][0][1]
165-
assert client_order_id.startswith("x-HNA2TXFJ")
165+
assert client_order_id.startswith("x-TKT5PX2F")
166166

167167
m.post(
168168
"https://api.binance.com/api/v3/order",
@@ -185,7 +185,7 @@ async def test_spot_cancel_replace_id_async():
185185

186186
def handler(url, **kwargs):
187187
client_order_id = kwargs["data"][0][1]
188-
assert client_order_id.startswith("x-HNA2TXFJ")
188+
assert client_order_id.startswith("x-TKT5PX2F")
189189

190190
m.post(
191191
"https://api.binance.com/api/v3/order/cancelReplace",
@@ -205,7 +205,7 @@ async def test_swap_id_async():
205205
with aioresponses() as m:
206206

207207
def handler(url, **kwargs):
208-
assert "x-Cb7ytekJ" in kwargs["data"][0][1]
208+
assert "x-cvBPrNm9" in kwargs["data"][0][1]
209209

210210
url_pattern = re.compile(r"https://fapi\.binance\.com/fapi/v1/order")
211211
m.post(
@@ -225,7 +225,7 @@ async def test_swap_trigger_id_async():
225225
with aioresponses() as m:
226226

227227
def handler(url, **kwargs):
228-
assert "x-Cb7ytekJ" in kwargs["data"][1][1]
228+
assert "x-cvBPrNm9" in kwargs["data"][1][1]
229229

230230
url_pattern = re.compile(r"https://fapi\.binance\.com/fapi/v1/algoOrder")
231231
m.post(
@@ -246,7 +246,7 @@ async def test_swap_trigger_endpoint_id_async():
246246

247247
def handler(url, **kwargs):
248248
# print(kwargs["data"])
249-
assert "x-Cb7ytekJ" in kwargs["data"][1][1]
249+
assert "x-cvBPrNm9" in kwargs["data"][1][1]
250250

251251
url_pattern = re.compile(r"https://fapi\.binance\.com/fapi/v1/algoOrder")
252252
m.post(
@@ -267,7 +267,7 @@ async def test_papi_um_id_async():
267267

268268
def handler(url, **kwargs):
269269
client_order_id = kwargs["data"][0][1]
270-
assert client_order_id.startswith("x-Cb7ytekJ")
270+
assert client_order_id.startswith("x-cvBPrNm9")
271271

272272
m.post(
273273
"https://papi.binance.com/papi/v1/um/order",
@@ -288,7 +288,7 @@ async def test_papi_cm_id_async():
288288

289289
def handler(url, **kwargs):
290290
client_order_id = kwargs["data"][0][1]
291-
assert client_order_id.startswith("x-Cb7ytekJ")
291+
assert client_order_id.startswith("x-cvBPrNm9")
292292

293293
m.post(
294294
"https://papi.binance.com/papi/v1/cm/order",
@@ -309,7 +309,7 @@ async def test_coin_id_async():
309309

310310
def handler(url, **kwargs):
311311
client_order_id = kwargs["data"][0][1]
312-
assert client_order_id.startswith("x-Cb7ytekJ")
312+
assert client_order_id.startswith("x-cvBPrNm9")
313313

314314
m.post(
315315
"https://dapi.binance.com/dapi/v1/order",
@@ -330,7 +330,7 @@ async def test_spot_oco_id():
330330

331331
def handler(url, **kwargs):
332332
client_order_id = kwargs["data"][0][1]
333-
assert client_order_id.startswith("x-HNA2TXFJ")
333+
assert client_order_id.startswith("x-TKT5PX2F")
334334

335335
m.post(
336336
"https://api.binance.com/api/v3/orderList/oco",
@@ -350,7 +350,7 @@ async def test_swap_batch_id_async():
350350
clientAsync = AsyncClient(api_key="api_key", api_secret="api_secret")
351351

352352
def handler(url, **kwargs):
353-
assert "x-Cb7ytekJ" in kwargs["data"]
353+
assert "x-cvBPrNm9" in kwargs["data"]
354354

355355
m.post(
356356
"https://fapi.binance.com/fapi/v1/batchOrders",
@@ -370,7 +370,7 @@ async def test_coin_batch_id_async():
370370
clientAsync = AsyncClient(api_key="api_key", api_secret="api_secret")
371371

372372
def handler(url, **kwargs):
373-
assert "x-Cb7ytekJ" in kwargs["data"][0][1]
373+
assert "x-cvBPrNm9" in kwargs["data"][0][1]
374374

375375
m.post(
376376
"https://dapi.binance.com/dapi/v1/batchOrders",

0 commit comments

Comments
 (0)