fixup! GateIO: Split asset.Futures into CoinM and USDT

gbjk · Jan 21, 2025 · 876972b · 876972b
1 parent 791a730
commit 876972b
Showing 1 changed file with 13 additions and 41 deletions.
diff --git a/exchanges/gateio/gateio_test.go b/exchanges/gateio/gateio_test.go
@@ -1891,47 +1891,19 @@ func TestGetActiveOrders(t *testing.T) {
 
 func TestGetOrderHistory(t *testing.T) {
 	sharedtestvalues.SkipTestIfCredentialsUnset(t, g)
-	var multiOrderRequest = order.MultiOrderRequest{
-		Type:      order.AnyType,
-		AssetType: asset.Spot,
-		Side:      order.Buy,
-	}
-	enabledPairs, err := g.GetEnabledPairs(asset.Spot)
-	if err != nil {
-		t.Fatal(err)
-	}
-	multiOrderRequest.Pairs = enabledPairs[:3]
-	_, err = g.GetOrderHistory(context.Background(), &multiOrderRequest)
-	if err != nil {
-		t.Errorf("%s GetOrderhistory() error: %v", g.Name, err)
-	}
-	multiOrderRequest.AssetType = asset.Futures
-	multiOrderRequest.Pairs, err = g.GetEnabledPairs(asset.Futures)
-	if err != nil {
-		t.Fatal(err)
-	}
-	multiOrderRequest.Pairs = multiOrderRequest.Pairs[len(multiOrderRequest.Pairs)-4:]
-	_, err = g.GetOrderHistory(context.Background(), &multiOrderRequest)
-	if err != nil {
-		t.Errorf("%s GetOrderhistory() error: %v", g.Name, err)
-	}
-	multiOrderRequest.AssetType = asset.DeliveryFutures
-	multiOrderRequest.Pairs, err = g.GetEnabledPairs(asset.DeliveryFutures)
-	if err != nil {
-		t.Fatal(err)
-	}
-	_, err = g.GetOrderHistory(context.Background(), &multiOrderRequest)
-	if err != nil {
-		t.Errorf("%s GetOrderhistory() error: %v", g.Name, err)
-	}
-	multiOrderRequest.AssetType = asset.Options
-	multiOrderRequest.Pairs, err = g.GetEnabledPairs(asset.Options)
-	if err != nil {
-		t.Fatal(err)
-	}
-	_, err = g.GetOrderHistory(context.Background(), &multiOrderRequest)
-	if err != nil {
-		t.Errorf("%s GetOrderhistory() error: %v", g.Name, err)
+	for _, a := range g.GetAssetTypes(false) {
+		enabledPairs := getPairs(t, a)
+		if len(enabledPairs) > 4 {
+			enabledPairs = enabledPairs[:4]
+		}
+		var multiOrderRequest = order.MultiOrderRequest{
+			Type:      order.AnyType,
+			Side:      order.Buy,
+			Pairs:     enabledPairs,
+			AssetType: a,
+		}
+		_, err := g.GetOrderHistory(context.Background(), &multiOrderRequest)
+		assert.NoError(t, err, "GetOrderHistory should not error for %s", a)
 	}
 }