Testare le classi toString, clone ed equals [JUnit]

[Bombonera]

Devo testare questi metodi. Precedentemente ho creato una classe che testa il metodo Fibonacci. Ad esempio:

@Test
assertEquals(1, fibonacci(0));

come potrei fare con il metodo toString, equals e clone utilizzando JUnit ?

[andbin]

Es.

public void testTwoIntegersEqual() {
    assertTrue(new Integer(123).equals(new Integer(123)));
}

public void testTwoIntegersNotEqual() {
    assertFalse(new Integer(123).equals(new Integer(12345)));
}

public void testIntegerNotEqualToOtherType() {
    assertFalse(new Integer(123).equals("xyz"));
}

[Bombonera]

Ho provato a creare un metodo che testa il metodo toString della classe Targa.

@Test
public void testToString()
{
	String toString = targa1.toString();
	assertTrue(toString.contains("Targa[c1 = " + targa1.c1 + "; c2 = "+ targa1.c2 + "; c3 = " + targa1.c3 +"]"));
}

il test mi da esito negativo

[andbin]

il test mi da esito negativo

Come è fatto il toString() di Targa? Dove/come assegni quel targa1?
E comunque non va bene che usi targa1.c1 ecc...

[Bombonera]

Come è fatto il toString() di Targa? Dove/come assegni quel targa1?
E comunque non va bene che usi targa1.c1 ecc...

public String toString()
{
return "Targa[c1 = " + c1 + "; c2 = "+ c2 + "; c3 = " + c3 +"]";
}

[Alex'87]

E cosa restituisce?

Un consiglio: invece di assertTrue usa assertEquals: in questo modo nel messaggio di errore ottieni qualche info in più:

String expectedValue = "PIPPO";
String actualValue = "PIPPA";

assertEquals(expectedValue, actualValue);

// l'errore dirà una cosa del tipo "expected PIPPO, actual value is PIPP[A]"

Comunque accedere dall'esterno direttamente a c1, c2 ecc è terribile D:

[Bombonera]

E cosa restituisce?

Un consiglio: invece di assertTrue usa assertEquals: in questo modo nel messaggio di errore ottieni qualche info in più:

String expectedValue = "PIPPO";
String actualValue = "PIPPA";

assertEquals(expectedValue, actualValue);

// l'errore dirà una cosa del tipo "expected PIPPO, actual value is PIPP[A]"

Comunque accedere dall'esterno direttamente a c1, c2 ecc è terribile D:

E' corretto testarla cosi ?

@Test
public void testToString()
{
     Targa targa = new Targa("hgt", "455", "jjj");
     assertEquals("Targa[c1 = " + targa.c1 + "; c2 = "+ targa.c2 + "; c3 = " + targa.c3 +"]",     targa.toString());
}

oppure direttamente

Targa targa =new Targa("hgt","455","jjj");
assertEquals("Targa[c1 = hgt; c2 = 455; c3 = jjj]", targa.toString());

il test mi da esito positivo. Ma è corretta la forma ?

[andbin]

oppure direttamente

Targa targa =new Targa("hgt","455","jjj");
assertEquals("Targa[c1 = hgt; c2 = 455; c3 = jjj]", targa.toString());

Ma è corretta la forma ?

Sì, è corretta la forma. Ed è meglio dell'altra dove usi targa.c1 ecc... E ti spiego anche perché. Metti caso che il costruttore di Targa sia sballato, metti che sia "saltato" l'assegnamento ad un campo, che pertanto rimane a null. Siccome il toString usa quel campo e pure la concatenazione nel tuo test usa quel campo, avresti comunque un match ma testeresti una cosa fasulla ...