Skip to content

Commit e231104

Browse files
localai-botmudler
andauthored
fix(mcp): drop the duplicated scheduling methods on stubClient (#11323)
master does not compile: vet: core/http/endpoints/mcp/localai_assistant_test.go:157:19: method stubClient.ListScheduling already declared at core/http/endpoints/mcp/localai_assistant_test.go:87:19 Two fixes for the same breakage landed. The four Scheduling methods were already present at lines 87-99, in interface order after ListNodes, by the time #11318 merged; #11318 appended its own copy after GetRouterDecisions. The two blocks sit in different parts of the file, so git merged both without a conflict and nothing flagged it. Remove the appended copy and keep the one in interface order. Pure deletion, no behaviour change. Verified: go vet clean on ./core/http/endpoints/mcp/, and go test ./core/http/endpoints/mcp/ passes. Assisted-by: Claude Code:claude-opus-5 [Read] [Edit] [Bash] Signed-off-by: Ettore Di Giacinto <mudler@localai.io> Co-authored-by: Ettore Di Giacinto <mudler@localai.io>
1 parent 6bdb04a commit e231104

1 file changed

Lines changed: 0 additions & 16 deletions

File tree

core/http/endpoints/mcp/localai_assistant_test.go

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -154,22 +154,6 @@ func (stubClient) GetRouterDecisions(_ context.Context, _ localaitools.RouterDec
154154
return []localaitools.RouterDecision{}, nil
155155
}
156156

157-
func (stubClient) ListScheduling(_ context.Context) ([]localaitools.ModelSchedulingConfig, error) {
158-
return []localaitools.ModelSchedulingConfig{}, nil
159-
}
160-
161-
func (stubClient) GetScheduling(_ context.Context, _ string) (*localaitools.ModelSchedulingConfig, error) {
162-
return &localaitools.ModelSchedulingConfig{}, nil
163-
}
164-
165-
func (stubClient) SetScheduling(_ context.Context, _ localaitools.SetSchedulingRequest) (*localaitools.ModelSchedulingConfig, error) {
166-
return &localaitools.ModelSchedulingConfig{}, nil
167-
}
168-
169-
func (stubClient) DeleteScheduling(_ context.Context, _ string) error {
170-
return nil
171-
}
172-
173157
var _ = Describe("LocalAIAssistantHolder", func() {
174158
var ctx context.Context
175159

0 commit comments

Comments
 (0)