sort exchanges based on what is tested first

plugins/factory.go

@@ -193,26 +193,32 @@ func loadExchanges() {
 		},
 	}
 
-	// add all CCXT exchanges
-	sortOrderOffset := len(*exchanges)
-	for i, exchangeName := range sdk.GetExchangeList() {
-		key := fmt.Sprintf("ccxt-%s", exchangeName)
-		_, tested := testedCcxtExchanges[exchangeName]
-		boundExchangeName := exchangeName
+	// add all CCXT exchanges (tested exchanges first)
+	sortOrderIndex := len(*exchanges)
+	for _, t := range []bool{true, false} {
+		for _, exchangeName := range sdk.GetExchangeList() {
+			key := fmt.Sprintf("ccxt-%s", exchangeName)
+			_, tested := testedCcxtExchanges[exchangeName]
+			if tested != t {
+				continue
+			}
+			boundExchangeName := exchangeName
 
-		(*exchanges)[key] = ExchangeContainer{
-			SortOrder:    uint16(i + sortOrderOffset),
-			Description:  exchangeName + " is automatically added via ccxt-rest",
-			TradeEnabled: true,
-			Tested:       tested,
-			makeFn: func(exchangeFactoryData exchangeFactoryData) (api.Exchange, error) {
-				return makeCcxtExchange(
-					boundExchangeName,
-					nil,
-					exchangeFactoryData.apiKeys,
-					exchangeFactoryData.simMode,
-				)
-			},
+			(*exchanges)[key] = ExchangeContainer{
+				SortOrder:    uint16(sortOrderIndex),
+				Description:  exchangeName + " is automatically added via ccxt-rest",
+				TradeEnabled: true,
+				Tested:       tested,
+				makeFn: func(exchangeFactoryData exchangeFactoryData) (api.Exchange, error) {
+					return makeCcxtExchange(
+						boundExchangeName,
+						nil,
+						exchangeFactoryData.apiKeys,
+						exchangeFactoryData.simMode,
+					)
+				},
+			}
+			sortOrderIndex++
 		}
 	}
 }