[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. (tanto se sbaglio ci sarà Cdimauro a tirarmi le orecchie )

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