search

GetnaG / ing-sw-2019-abbo-attolini-bentej

Progetto di Ingegneria del Software 2019
0 stars 1 forks source link

Unit Test very little test code #2

Closed ingconti closed 5 years ago

ingconti commented 5 years ago

for now You have not reached deadline. You covered very few cases.

poligiulio commented 5 years ago

La ringrazio per la segnalazione; a riguardo le chiedo un chiarimento. Intende dire che i test creati non coprono abbastanza casi o che il numero di classi di test (in src/test) è di molto inferiore al numero delle altre classi (in src/main)? Nel secondo caso potrebbe essere perché, per avere un codice compilabile, abbiamo creato delle classi "vuote" che fanno da segnaposto in attesa che finiamo di implementarle? Giusto ieri a laboratorio ci è stato detto che andava tutto bene, non mi aspettavo un issue, tanto meno mi aspettavo di non aver raggiunto la deadline. Spero che possa indirizzarci verso la soluzione, per raggiungere la deadline al più presto. Buona serata

ingconti commented 5 years ago

Placeholders are fine, at start. PLS add a richer set of tests, with some explanation (java doc)

I will revise them and raise evaluation

ingconti commented 5 years ago

ddi You test for example:

FrenzyPlayerBoard. thx

poligiulio commented 5 years ago

Thank you, we will add some javadoc to the test classes too.

FrenzyPlayerBoard is one of those placeholders, so we have not written a test class for it yet. Are we required to write the test classes before the actual ones? (Test driven development)

ingconti commented 5 years ago

The plain, straight way (as we showed in labs... and suggested to many..) was to use intellj automated test generation AND FILLL THEM.

It appears a bit strange to me to have, say, 30 classes and 2 files for tests, even if accettaptable, seems weird.

(obviously You have to fill at least partially, those methods..)