SoftwareSynthesis / SoftwareEngineeringProject

Progetto per lo sviluppo del software di comunzione MyTalk relativo al corso di Ingegneria del Software (Laurea Triennale Universita' degli Studi di Padova)
0 stars 0 forks source link

[PushInbound] stato e disconnessione #272

Closed diegoberaldin closed 11 years ago

diegoberaldin commented 11 years ago

L'informale resoconto della verifica di questa classe è che ho modificato nel ramo 5 del mega-branch il fatto che si manda la rappresentazione in stringa dello stato in formato maiuscolo, dal momento che il client se lo aspetta in minuscolo.

diegoberaldin commented 11 years ago

p.p.s. Altro problema rilevato (non coi test perché non posso farlo) ma con analisi statica e un po' di debugger... Cosa succede sempre nel ramo 5 quando si trova un contatto nella rubrica che però non è connesso? In quel caso il mio caro metodo statico findClient(...) ritorna correttamente null ma prima di farse sendTo.getWsOutbound() non si fa alcun controllo per verificare se sentTo sia null. La cosa a mio avviso dovrebbe essere gestita...

diegoberaldin commented 11 years ago

p.p.p.s. (chiedo scusa se arrivano a sprazzi i miei commenti) Mi confermi che nel ramo 6 deve essere fatto solo quello? Rimandare a chi mi ha inviato il messaggio di testo 6|?

marcoskivo commented 11 years ago

ho sistemato e aggiunto anche la chat.. non so se devi modificare dei test..

diegoberaldin commented 11 years ago

Sì dovrò farlo, provvedo entro lunedì a risistemare il tutto!