[TASK 8.1.2] Pair Programming: Cdimauro vs Bonfo

[Bonfo]

Task 8.1.2:
Evidenziare la gemma pivot di un GemsPair facendola pulsare. La pulsazione deve terminare una volta che la coppia si spezza, cioè quando una delle due gemme collide.

E' la mia prima test list: siate buoni. :flower: (tanto se sbaglio ci sarà Cdimauro a tirarmi le orecchie :D)

1) testare che GemsPair non deve pulsare se è costituita da 0 gemme
2) testare che GemsPair non deve pulsare se è costituita da 1 gemma pivot
3) testare che GemsPair non deve pulsare se è costituita da 1 gemma slave
4) testare che GemsPair sia costituita da 2 gemme, 1 pivot e 1 slave

Nel caso 4:
5) testare che GemsPair non deve pulsare se la gemma pivot è ferma, ovvero non sta cadendo
6) testare che GemsPair non deve pulsare se la gemma slave è ferma, ovvero non sta cadendo
7) testare che se entrambe le gemme di GemsPair stiano cadendo possa pulsare

[cdimauro]

Ed ecco che (finalmente) arriva il primo test:
package it.diamonds.tests;

public class TestGemsPairPulsation extends TestGemsPairBase
{

protected void setUp()
{
super.setUp();
}

public void testNoPivotAndNoSlaveGems()
{
getPairFromController().setNoPivotGem();
getPairFromController().setNoSlaveGem();
assertFalse(
"Pivot and slave gems are set-up",
getPairFromController().isPulsing());

}


}
Adesso devi scrivere il codice MINIMO necessario per farlo passare.

[Bonfo]

Ed ecco il mio codice....prima volta


public boolean isPulsing()
{
return false;
}


Test Ok

[cdimauro]

Ecco il codice del secondo test:
public void testNoSlaveGem()
{
getPairFromController().setNoSlaveGem();
assertFalse(
"Slave gem was set-up",
getPairFromController().isPulsing());
}

[cdimauro]

Qui Bonfo si è accorto che il test passa lo stesso senza cambiare il codice. Il che ci ha fatto insospettire sull'utilità di qualche test.
Più precisamente, è venuto fuori che sono sufficienti i test 2 e 3 per garantire che isPulsing faccia il suo lavoro soltanto se entrambe le gemme sono presenti in gemsPair.

Quindi procedo a scrivere il test3:

public void testNoPivotGem()
{
getPairFromController().setNoPivotGem();
assertFalse(
"Pivot gem was set-up",
getPairFromController().isPulsing());
}
Il codice del test1 verrà rimosso.

[cdimauro]

Anche così il test passa senza alcuna modifica. Quindi è chiaro che la nostra test base va ripensata. Buttiamo tutto il codice e torniamo all'origine.

Il task dice che la gemma pivot di gemspair deve pulsare fino a quando la coppia non si spezza. Quindi testiamo per prima il caso che pulsi quando la coppia è unita:
public void testWithPivotAndSlaveGems()
{
assertTrue(
"Pivot and slave gems aren't set-up",
getPairFromController().isPulsing());
}
A te la mano.

[Bonfo]

Dualmente a prima, ecco il codice minimo:

public boolean isPulsing()
{
return true;
}


Vediamo ora come si procede.
Prossimo test

[cdimauro]

Riprendiamo il test2:
public void testNoSlaveGem()
{
getPairFromController().setNoSlaveGem();
assertFalse(
"Slave gem was set-up",
getPairFromController().isPulsing());
}

[Bonfo]

Modifica al metodo isPulsing()


public boolean isPulsing()
{
if(getSlaveGem()==null)
{
return false;
}
return true;
}


Green

Sotto il prossimo...

[cdimauro]

Riprendiamo anche il test3:
public void testNoPivotGem()
{
getPairFromController().setNoPivotGem();
assertFalse(
"Pivot gem was set-up",
getPairFromController().isPulsing());
}

[Bonfo]

Seconda modifica al metodo isPulsing()

public boolean isPulsing()
{
if(getSlaveGem()==null || getPivotGem()==null)
{
return false;
}
return true;
}


Green

[cdimauro]

Dobbiamo testare che se la gemma pivot non sta cadendo, non deve nemmeno pulsare:
public void testPivotIsNotFalling()
{
getGridController().setDelay(300);
getGridFromController().insertGem(2, 4, Gem.diamond(3500));
getGridController().update(getTimer());
assertFalse(
"Pivot gem was falling down",
getPairFromController().isPulsing());
}

[Bonfo]

Ulteriore modifica la metodo isPulsing() più un piccolo refactoring

public boolean isPulsing()
{
return getPivotGem()!=null && getSlaveGem()!=null && getPivotGem().isFalling();
}


Green

E via col prossimo ;)

[cdimauro]

Ecco il codice per testare che se anche la gemma slave non sta cadendo, quella pivot non deve pulsare:
public void testSlaveIsNotFalling()
{
getGridController().setDelay(300);
Gem gem = Gem.diamond(3500);
getGridFromController().insertGem(1, 3, gem);
getPairFromController().setPivotGem(gem);
getGridFromController().insertGem(2, 3, Gem.diamond(3500));
getGridController().update(getTimer());
assertFalse(
"Pivot gem was falling down",
getPairFromController().isPulsing());
}

[Bonfo]

Ennesima modifica a isPulsing()

public boolean isPulsing()
{
return getPivotGem()!=null && getSlaveGem()!=null && getPivotGem().isFalling() && getSlaveGem().isFalling();
}


Green

[cdimauro]

L'ultimo test che rimane è inutile, perché i test precedenti ci garantiscono che gemsPair deve avere per forza pivot e slave, e che entrambi debbano cadere.

Non rimane che implementare il codice per far pulsare la gemma Pivot.

[Bonfo]

Bene...rieccoci qua.
Cdimauro non può testare perchè ha ant che non funziona.
Sta seguendo passso passo con me.

Stiamo modificando Gem perchè esponga i metodi per la pulsazione.
Ecco i test:

private Gem createGem()
{
return Gem.diamond(3500);
}

public void testStartPulsing()
{
Gem gem=createGem();
gem.startPulsing();
assertTrue(gem.isPulsing());
}

public void testStopPulsing()
{
Gem gem=createGem();
gem.startPulsing();
assertTrue(gem.isPulsing());


gem.stopPulsing();
assertFalse(gem.isPulsing());
}

[Bonfo]

Ed ecco il codice per passare i test

public void startPulsing()
{
sprite.setSizeMultiplier(4);
sprite.setSpeedDivisor(10);
sprite.startPulsing();
}

public void stopPulsing()
{
sprite.stopPulsing();
}

public boolean isPulsing()
{
return sprite.isPulsing();
}


Green

[cdimauro]

Aggiungiamo anche un test per controllare che la gemma pivot pulsi se lo fa gemsPair:
public void testPivotGemPulsingWhenGemsPairPulsing()
{
assertEquals(
"Pivot and GemsPair aren't pulsing together",
getPairFromController().isPulsing(),
getPairFromController().getPivotGem().isPulsing());
}

[Bonfo]

Aggiunto il metodo privato updatePulsingState()


private void updatePulsingState()
{
if(getPivotGem()==null)
{
return;
}

if(isPulsing())
{
if(!getPivotGem().isPulsing())
{
getPivotGem().startPulsing();
}
}
else
{
if(getPivotGem().isPulsing())
{
getPivotGem().stopPulsing();
}
}
}


...ed utilizzato per aggiornare lo stato della pulsazione in:
- public void setPivotGem(Gem gem)
- public void setNoPivotGem()
- public void setNoSlaveGem()
- public void update(TimerInterface timer)

Green

[cdimauro]

Adesso tocca aggiungere la possibilità di settare divisore e moltiplicatore per il pulsing di una gemma.

Scrivo i test, ripresi pari pari da quelli di Sprite:
public void testSetSpeedDivisor()
{
assertEquals(10f, gem.getSpeedDivisor());
gem.setSpeedDivisor(3.14f);
assertEquals(3.14f, gem.getSpeedDivisor());
}

public void testSetSizeMultiplier()
{
assertEquals(4f, gem.getSizeMultiplier());
gem.setSizeMultiplier(2.71f);
assertEquals(2.71f, gem.getSizeMultiplier());
}

[fek]

Potete evitare i float ed esprimere i valori in percentuale per favore?

[Bonfo]

Aggiunti i metodi a Gem:

public void setSpeedDivisor(float speedDivisor)
{
sprite.setSpeedDivisor(speedDivisor);
}

public float getSpeedDivisor()
{
return sprite.getSpeedDivisor();
}

public void setSizeMultiplier(float sizeMultiplier)
{
sprite.setSizeMultiplier(sizeMultiplier);
}

public float getSizeMultiplier()
{
return sprite.getSizeMultiplier();
}


Green

[cdimauro]

Potete evitare i float ed esprimere i valori in percentuale per favore?
Il fatto è che il codice di Sprite, test inclusi, usano float.

Io preferirei finire il codice per adesso, e poi magari lasciare il refactoring alla fine. Anche perché i miei neuroni stanno cominciando a cedere causa sonno arretrato. :p

P.S. Sai che preferisco lavorare con gli interi, piuttosto che coi float, di cui avrei fatto volentieri a meno fin dall'inizio. :D

[fek]

Il fatto è che il codice di Sprite, test inclusi, usano float.

Io preferirei finire il codice per adesso, e poi magari lasciare il refactoring alla fine. Anche perché i miei neuroni stanno cominciando a cedere causa sonno arretrato. :p

Oki, ci penso io allora :)

[cdimauro]

Valerio è dovuto andare via. Per cui non abbiamo potuto finire il lavoro.

Però la build è verde e il pulsing è implementato correttamente (almeno sembra).

Magari stasera chi vuole fa un po' di refactoring, e poi domani sera alle 18 riprendiamo con Valerio per finire il passaggio dei valori da config a GemsPair.

Io vado a mettere a riposo le ossa. :fagiano:

[cdimauro]

Per completare il lavoro e permettere il passaggio dei parametri da GemsPair alla gemma Pivot, approfittiamo del fatto che GemsPair possa accedere all'istanza di Grid, il cui costruttore prevede come parametro un'istanza della classe Config.

Sfortunatamente Grid usa l'istanza di Config e la butta via, per cui si rende necessario memorizzarla e fornire un metodo pubblico getConfig() che la ritorni all'occorrenza. Nuovo metodo -> relativo test, per cui ne scrivo il codice:
public void testGetConfig()
{
Config config = Config.createForTesting();
grid = new Grid(
new Rectangle(40, 40, 295, 487), config);
assertEquals(grid.getConfig(), config);
}
Passo la palla a Valerio per scrivere il codice per farlo passare.

[Bonfo]

Non difficile...
aggiunto il campo privato

private Config config;

ed il metodo per potervi accedere

public Config getConfig()
{
return config;
}

ovviamente config viene settato nel costruttore... ;)

Green

[Bonfo]

Spostai i metodi setSpeedDivisor e setSizeMultiplier da gem.starPulsing() in setPivotGem di GemsPair e sotituiti con:

pivotGem.setSpeedDivisor(grid.getConfig().getInteger("SpeedDivisor"));
pivotGem.setSizeMultiplier(grid.getConfig().getInteger("SizeMultiplier"));


Green

TASK COMPLETATO :winner: