exchanges/wrappers: shift bespoke fetch ticker, orderbook and accountinfo

cmd/exchange_wrapper_standards/exchange_wrapper_standards_test.go

@@ -40,7 +40,7 @@
 // singleExchangeOverride enter an exchange name to only test that exchange
 var singleExchangeOverride = ""
 
 func TestAllExchangeWrappers(t *testing.T) {
 	t.Parallel()
 	cfg := config.GetConfig()
 	err := cfg.LoadConfig("../../testdata/configtest.json", true)
@@ -50,7 +50,6 @@
 	for i := range cfg.Exchanges {
 		name := strings.ToLower(cfg.Exchanges[i].Name)
 		t.Run(name+" wrapper tests", func(t *testing.T) {
-			t.Parallel()
 			if common.StringDataContains(unsupportedExchangeNames, name) {
 				t.Skipf("skipping unsupported exchange %v", name)
 			}
@@ -62,7 +61,7 @@
 				// rather than skipping tests where execution is blocked, provide an expired
 				// context, so no executions can take place
 				var cancelFn context.CancelFunc
-				ctx, cancelFn = context.WithTimeout(context.Background(), 0)
+				ctx, cancelFn = context.WithTimeout(ctx, 0)
 				cancelFn()
 			}
 			exch, assetPairs := setupExchange(ctx, t, name, cfg)
@@ -197,16 +196,8 @@
 
 func handleExchangeWrapperTests(ctx context.Context, t *testing.T, actualExchange reflect.Value, methodNames []string, exch exchange.IBotExchange, assetParams []assetPair, groupTestID string) {
 	t.Helper()
-	if groupTestID == "PRIORITY GROUP" {
-		// Some update requests check for authenticated support, but we don't
-		// want to test them. This will fail fetcher functions in secondary
-		// group.
-		exch.GetBase().API.AuthenticatedSupport = false
-	} else {
-		exch.GetBase().API.AuthenticatedSupport = true
-	}
-
 	t.Run(groupTestID, func(t *testing.T) {
+		t.Parallel()
 		for x := range methodNames {
 			method := actualExchange.MethodByName(methodNames[x])
 

exchanges/kucoin/kucoin_wrapper.go

@@ -415,11 +415,7 @@ func (ku *Kucoin) UpdateOrderbook(ctx context.Context, pair currency.Pair, asset
 	case asset.Futures:
 		ordBook, err = ku.GetFuturesOrderbook(ctx, pair.String())
 	case asset.Spot, asset.Margin:
-		if ku.IsRESTAuthenticationSupported() && ku.AreCredentialsValid(ctx) {
-			ordBook, err = ku.GetOrderbook(ctx, pair.String())
-		} else {
-			ordBook, err = ku.GetPartOrderbook100(ctx, pair.String())
-		}
+		ordBook, err = ku.GetPartOrderbook100(ctx, pair.String())
 	default:
 		return nil, fmt.Errorf("%w %v", asset.ErrNotSupported, assetType)
 	}