Function TestLoadConfigValid

cli/mcp/mcp_test.go:256–301

(t *testing.T)

}

func TestLoadConfigValid(t *testing.T) {
	dir := t.TempDir()
	cfgPath := filepath.Join(dir, "mcp.json")
	cfg := `{
		"mcpServers": [
			{"name":"srv1","command":"echo","transport":"stdio","enabled":true},
			{"name":"srv2","command":"cat","transport":"stdio","enabled":false},
			{"name":"srv3","command":"test","transport":"sse","url":"http://localhost:8080","enabled":true}
		]
	}`
	if err := os.WriteFile(cfgPath, []byte(cfg), 0644); err != nil {
		t.Fatal(err)
	}

	m := NewManager(testLogger())
	if err := m.LoadConfig(cfgPath); err != nil {
		t.Fatal(err)
	}

	// Only enabled servers should be loaded
	if len(m.servers) != 2 {
		t.Fatalf("got %d servers, want 2 (only enabled)", len(m.servers))
	}
	if _, ok := m.servers["srv1"]; !ok {
		t.Error("expected srv1")
	}
	if _, ok := m.servers["srv2"]; ok {
		t.Error("srv2 is disabled, should not be loaded")
	}
	if _, ok := m.servers["srv3"]; !ok {
		t.Error("expected srv3")
	}
	if m.servers["srv3"].Config.Transport != TransportSSE {
		t.Errorf("srv3 transport = %q, want sse", m.servers["srv3"].Config.Transport)
	}
	// Servers should be registered as Starting=true so /mcp status shows
	// "iniciando…" while StartAll runs in the background.
	for name, conn := range m.servers {
		if !conn.Status.Starting {
			t.Errorf("server %q: Status.Starting=false, want true after LoadConfig", name)
		}
		if conn.Status.Connected {
			t.Errorf("server %q: Status.Connected=true before StartAll", name)
		}
	}
}

// TestStartAllRecordsLastError verifies that a server failing to start
// surfaces the error on Status.LastError (not just in logs) so /mcp