Bad assertion in testCallTool

Descrizione

In AbstractMcpSyncClientTests, there is a test case called {testCallTool](https://github.com/modelcontextprotocol/java-sdk/blob/110a8d1940ca2edf45524332dcc84b606e52d879/mcp/src/test/java/io/modelcontextprotocol/client/AbstractMcpSyncClientTests.java#L238)

on line 247 is this assertion:

			assertThat(result.isError()).isNull();

Although currently satisfied by the 'everything server' (assertion passes with javascript server) this assumes that the result.isError() (a Boolean value) is null if there is no error. Since 'isError() is set to 'false' and not null on success (by java server at least) this line should probably be:

			assertThat(result.isError()).isFalse();

and that the everything server should be fixed to return isError=false rather than isError=null as it does currently.

Come iniziare

Leggi tutta la issue e poi la guida ai contributi del progetto.

Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.

Fai un fork del repository e lavora su un branch.

Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Inizia in mcp/src/test/java/io/modelcontextprotocol/client/AbstractMcpSyncClientTests.java, in testCallTool e nella relativa asserzione intorno alla riga 247. Controlla anche la risposta di successo del everything server. Il lavoro è completato quando i risultati riusciti assert isError() is false e il everything server restituisce false invece di null.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico

java, javascript

Ambito

testing

Tipo di issue

Bug

Difficoltà

2/5

Tempo stimato

1-3 ore

Stato di attività

Tranquilla

Chiarezza

Specificata chiaramente

Idoneità per principianti

75/100

Descrizione

Guida per i contributori

Come iniziare

Direzione di ricerca

Valutazione

Ricevi le nuove issue nella tua casella