Fix tests by increasing limits to 1000

0b100100 · web-flow · commit cebb4e3a0ebc · 2022-02-12T15:22:02.000+01:00
Fix the historical kline tests by setting all limits to 1000.
diff --git a/tests/test_historical_klines.py b/tests/test_historical_klines.py
@@ -56,11 +56,11 @@ def test_exact_amount():
             json=first_available_res,
         )
         m.get(
-            "https://api.binance.com/api/v3/klines?interval=1m&limit=500&startTime=1519862400000&symbol=BNBBTC",
+            "https://api.binance.com/api/v3/klines?interval=1m&limit=1000&startTime=1519862400000&symbol=BNBBTC",
             json=first_res,
         )
         m.get(
-            "https://api.binance.com/api/v3/klines?interval=1m&limit=500&startTime=1519892400000&symbol=BNBBTC",
+            "https://api.binance.com/api/v3/klines?interval=1m&limit=1000&startTime=1519892400000&symbol=BNBBTC",
             json=second_res,
         )
         klines = client.get_historical_klines(
@@ -113,7 +113,7 @@ def test_start_and_end_str():
             json=first_available_res,
         )
         m.get(
-            "https://api.binance.com/api/v3/klines?interval=1m&limit=500&startTime=1519862400000&endTime=1519880400000&symbol=BNBBTC",
+            "https://api.binance.com/api/v3/klines?interval=1m&limit=1000&startTime=1519862400000&endTime=1519880400000&symbol=BNBBTC",
             json=first_res,
         )
         klines = client.get_historical_klines(
@@ -169,7 +169,7 @@ def test_start_and_end_timestamp():
             json=first_available_res,
         )
         m.get(
-            "https://api.binance.com/api/v3/klines?interval=1m&limit=500&startTime=1519862400000&endTime=1519880400000&symbol=BNBBTC",
+            "https://api.binance.com/api/v3/klines?interval=1m&limit=1000&startTime=1519862400000&endTime=1519880400000&symbol=BNBBTC",
             json=first_res,
         )
         klines = client.get_historical_klines(
@@ -225,7 +225,7 @@ def test_historical_kline_generator():
             json=first_available_res,
         )
         m.get(
-            "https://api.binance.com/api/v3/klines?interval=1m&limit=500&startTime=1519862400000&endTime=1519880400000&symbol=BNBBTC",
+            "https://api.binance.com/api/v3/klines?interval=1m&limit=1000&startTime=1519862400000&endTime=1519880400000&symbol=BNBBTC",
             json=first_res,
         )
         klines = client.get_historical_klines_generator(
@@ -268,7 +268,7 @@ def test_historical_kline_generator_empty_response():
             json=first_available_res,
         )
         m.get(
-            "https://api.binance.com/api/v3/klines?interval=1m&limit=500&startTime=1519862400000&endTime=1519880400000&symbol=BNBBTC",
+            "https://api.binance.com/api/v3/klines?interval=1m&limit=1000&startTime=1519862400000&endTime=1519880400000&symbol=BNBBTC",
             json=first_res,
         )
         klines = client.get_historical_klines_generator(

Original file line number	Diff line number	Diff line change
`@@ -56,11 +56,11 @@ def test_exact_amount():`
`56`	`56`	`json=first_available_res,`
`57`	`57`	`)`
`58`	`58`	`m.get(`
`59`		`- "https://api.binance.com/api/v3/klines?interval=1m&limit=500&startTime=1519862400000&symbol=BNBBTC",`
	`59`	`+ "https://api.binance.com/api/v3/klines?interval=1m&limit=1000&startTime=1519862400000&symbol=BNBBTC",`
`60`	`60`	`json=first_res,`
`61`	`61`	`)`
`62`	`62`	`m.get(`
`63`		`- "https://api.binance.com/api/v3/klines?interval=1m&limit=500&startTime=1519892400000&symbol=BNBBTC",`
	`63`	`+ "https://api.binance.com/api/v3/klines?interval=1m&limit=1000&startTime=1519892400000&symbol=BNBBTC",`
`64`	`64`	`json=second_res,`
`65`	`65`	`)`
`66`	`66`	`klines = client.get_historical_klines(`
`@@ -113,7 +113,7 @@ def test_start_and_end_str():`
`113`	`113`	`json=first_available_res,`
`114`	`114`	`)`
`115`	`115`	`m.get(`
`116`		`- "https://api.binance.com/api/v3/klines?interval=1m&limit=500&startTime=1519862400000&endTime=1519880400000&symbol=BNBBTC",`
	`116`	`+ "https://api.binance.com/api/v3/klines?interval=1m&limit=1000&startTime=1519862400000&endTime=1519880400000&symbol=BNBBTC",`
`117`	`117`	`json=first_res,`
`118`	`118`	`)`
`119`	`119`	`klines = client.get_historical_klines(`
`@@ -169,7 +169,7 @@ def test_start_and_end_timestamp():`
`169`	`169`	`json=first_available_res,`
`170`	`170`	`)`
`171`	`171`	`m.get(`
`172`		`- "https://api.binance.com/api/v3/klines?interval=1m&limit=500&startTime=1519862400000&endTime=1519880400000&symbol=BNBBTC",`
	`172`	`+ "https://api.binance.com/api/v3/klines?interval=1m&limit=1000&startTime=1519862400000&endTime=1519880400000&symbol=BNBBTC",`
`173`	`173`	`json=first_res,`
`174`	`174`	`)`
`175`	`175`	`klines = client.get_historical_klines(`
`@@ -225,7 +225,7 @@ def test_historical_kline_generator():`
`225`	`225`	`json=first_available_res,`
`226`	`226`	`)`
`227`	`227`	`m.get(`
`228`		`- "https://api.binance.com/api/v3/klines?interval=1m&limit=500&startTime=1519862400000&endTime=1519880400000&symbol=BNBBTC",`
	`228`	`+ "https://api.binance.com/api/v3/klines?interval=1m&limit=1000&startTime=1519862400000&endTime=1519880400000&symbol=BNBBTC",`
`229`	`229`	`json=first_res,`
`230`	`230`	`)`
`231`	`231`	`klines = client.get_historical_klines_generator(`
`@@ -268,7 +268,7 @@ def test_historical_kline_generator_empty_response():`
`268`	`268`	`json=first_available_res,`
`269`	`269`	`)`
`270`	`270`	`m.get(`
`271`		`- "https://api.binance.com/api/v3/klines?interval=1m&limit=500&startTime=1519862400000&endTime=1519880400000&symbol=BNBBTC",`
	`271`	`+ "https://api.binance.com/api/v3/klines?interval=1m&limit=1000&startTime=1519862400000&endTime=1519880400000&symbol=BNBBTC",`
`272`	`272`	`json=first_res,`
`273`	`273`	`)`
`274`	`274`	`klines = client.get_historical_klines_generator(`