Call To Arms: oggi e domani si fa refactoring (ancora piu' del solito)

[^TiGeRShArK^]

Originariamente inviato da Ufo13

Vic a me non da errore sui test... Mi da errore perchè GridTestCase non contiene nessun test

a me con ant non da nessun errore, da eclipse mi da il tuo stesso prob con GridTestCase

[Ufo13]

ho fixato ora è green

[^TiGeRShArK^]

ok, confermo
qualche suggerimento per il metodo di cui sopra???

[VICIUS]

Originariamente inviato da Ufo13

Vic a me non da errore sui test... Mi da errore perchè GridTestCase non contiene nessun test

GridTestCase non viene eseguito perché non comincia con Test e va bene cosi. I test che non vanno sono commentati e stanno in TestBigGemInGrid.

ciao

[cionci]

Che si dice di bello ?!?!?

Sono tornato da poco...

[Ufo13]

Originariamente inviato da VICIUS

GridTestCase non viene eseguito perché non comincia con Test e va bene cosi. I test che non vanno sono commentati e stanno in TestBigGemInGrid.

ciao

aggiungendo:

testDummy() { assertTrue(true); }

va... Quindi mi sa che viene aggiunto anche se non inizia con Test...

per i test commentati ora guardo

[^TiGeRShArK^]

Originariamente inviato da cionci

Che si dice di bello ?!?!?

Sono tornato da poco...

bentornato tesoro!


per caso capisci qualcosa delle BigGem???
ci sono 4 test che non passano dopo un refactoring di vic e io non ho idea di come sono implementate le BigGem...

[Ufo13]

Originariamente inviato da ^TiGeRShArK^

bentornato tesoro!


per caso capisci qualcosa delle BigGem???
ci sono 4 test che non passano dopo un refactoring di vic e io non ho idea di come sono implementate le BigGem...

In pratica fallisce perchè quando vengono inserite le gemme vengono updatate e scendono di una posizione rendendo impossibile aggiungere quelle sotto.. Non capisco perchè prima funzionava ed adesso no :\

[cionci]

Originariamente inviato da ^TiGeRShArK^

bentornato tesoro!


per caso capisci qualcosa delle BigGem???
ci sono 4 test che non passano dopo un refactoring di vic e io non ho idea di come sono implementate le BigGem...

Ci posso provare... La build machine è verde, i test che falliscono sono commentati ?

[VICIUS]

Originariamente inviato da Ufo13

aggiungendo:

testDummy() { assertTrue(true); }

va... Quindi mi sa che viene aggiunto anche se non inizia con Test...

per i test commentati ora guardo

Strano ho provato più volte mentre mettevo a posto con ant e la classe non veniva mai chiamata.
Come lanci i test ?

ciao

[VICIUS]

Originariamente inviato da cionci

Ci posso provare... La build machine è verde, i test che falliscono sono commentati ?

Se guardi nei task di eclipse ci sono un po' di TODO da eliminare

ciao

[cionci]

Originariamente inviato da VICIUS

Se guardi nei task di eclipse ci sono un po' di TODO da eliminare

Ehm...dove ?

[VICIUS]

Originariamente inviato da cionci

Ehm...dove ?

Window -> Show view -> Other. In General scegli Tasks.

ciao

[Ufo13]

fatto io... Fixati i test...


Vicius: lanciando il tool grafico dentro eclipse mi dava quel errore...

[^TiGeRShArK^]

infatti anche a me con ant non dava problemi, ma con eclipse si senzaa testdummy.

Cmq buono che ora sia tutto verde.... qual'era il problema??

[Ufo13]

bisogna chiamare il gem.drop() quando si crea la bigGem in quei casi...

[VICIUS]

Stavo guardando i test che avevo sistemato poco fa e ho notato una marea di getGrid() getTimer() ma non riesco a capire a che cosa servono? per quale motivo mettere private gli attributi nella classe madre e usare delle getter per fare sto lavoro. Rendono praticamente illeggibile il codice a mio avviso.

ciao

[Bonfo]

A me hanno insegnato ad usare sempre dei getter...

...immagina che da domani invece di usare millisecondi usiamo i microsecondi per l'attributo pubblico la modifica toccherebbe l'intera classe con il getter basta fare:

return 100*time


Ti piace come risposta???

[cionci]

A parte che sono d'accordo con i getter, ma in ogni caso è una imposizione di checkstyle (quindi di fek ), non ci possono essere variabili pubbliche...a parte le costanti...

[Ufo13]

Originariamente inviato da VICIUS

Stavo guardando i test che avevo sistemato poco fa e ho notato una marea di getGrid() getTimer() ma non riesco a capire a che cosa servono? per quale motivo mettere private gli attributi nella classe madre e usare delle getter per fare sto lavoro. Rendono praticamente illeggibile il codice a mio avviso.

ciao

Lo so ma la build era rotta senza getter/setter. Comunque proprio pubblici era meglio di no... Al massimo protected ed era rotta anche così... CheckStyle li vuole privati o nisba :P