engine: Add additional SetupExchanges edge case test and prevent supp…

…ressed dry run warnings

engine/engine.go

@@ -854,11 +854,12 @@ func (bot *Engine) dryRunParamInteraction(param string) {
 		return
 	}
 
+	gctlog.Warnf(gctlog.Global,
+		"Command line argument '-%s' induces dry run mode."+
+			" Set -dryrun=false if you wish to override this.",
+		param)
+
 	if !bot.Settings.EnableDryRun {
-		gctlog.Warnf(gctlog.Global,
-			"Command line argument '-%s' induces dry run mode."+
-				" Set -dryrun=false if you wish to override this.",
-			param)
 		bot.Settings.EnableDryRun = true
 	}
 }
@@ -908,6 +909,10 @@ func (bot *Engine) SetupExchanges() error {
 		}
 	}
 
+	if bot.Settings.EnableAllExchanges && len(exchangesOverride) > 0 {
+		return errors.New("cannot enable all exchanges and specific exchanges concurrently")
+	}
+
 	var wg sync.WaitGroup
 	for x := range configs {
 		shouldLoad := false

engine/engine_test.go

@@ -430,7 +430,20 @@ func TestSetupExchanges(t *testing.T) {
 		assert.ErrorIs(t, e.SetupExchanges(), ErrNoExchangesLoaded)
 	})
 
-	// Test that adjusting settings induces dry run mode correctly
+	t.Run("EnableAllExchanges with specific exchanges set", func(t *testing.T) {
+		t.Parallel()
+		e := &Engine{
+			Config: &config.Config{},
+			Settings: Settings{
+				CoreSettings: CoreSettings{
+					EnableAllExchanges: true,
+					Exchanges:          "Bitstamp,Bitfinex",
+				},
+			},
+		}
+		assert.EqualError(t, e.SetupExchanges(), "cannot enable all exchanges and specific exchanges concurrently")
+	})
+
 	t.Run("Settings dry run toggling", func(t *testing.T) {
 		t.Parallel()
 		e := &Engine{