Se han modificado tanto los test asociados a la clase Persona como el...

Se han modificado tanto los test asociados a la clase Persona como el constructor de la clase para quitar de este un parámetro y ponerlo como atributo setteable de la clase, debido a especificaciones del enunciado

src/main/java/inf/uva/es/ivagonz/practica4/Persona.java

@@ -22,18 +22,17 @@ public class Persona {
 	 *            contenga números o caracteres extraños.
 	 * @param id
 	 *            Identificador de una Persona. Debe ser correcto: Positivo.
-	 * @param maxAmigos
-	 *            Maximo numero de personas que puede colar this. Debe ser
-	 *            correcto: positivo y menor de 10.
 	 * @param amigos
-	 *            Lista de amigos de this. Debe ser correcta: No nula.
+	 *            Lista de amigos de this. Debe ser correcta: No nula y no
+	 *            contener elementos nulos.
 	 * @param conocidos
-	 *            Lista de conocidos de this. Debe ser correcta: No nula.
+	 *            Lista de conocidos de this. Debe ser correcta: No nula y no
+	 *            contener elementos nulos.
 	 * @throws IllegalArgumentException
 	 *             si se incumple alguna de las condiciones impuestas a los
 	 *             parámetros.
 	 */
-	public Persona(String nombre, int id, int maxAmigos, ArrayList<Persona> amigos, ArrayList<Persona> conocidos) {
+	public Persona(String nombre, int id, ArrayList<Persona> amigos, ArrayList<Persona> conocidos) {
 		// TODO Auto-generated constructor stub
 	}
 
@@ -136,6 +135,18 @@ public class Persona {
 		return 0;
 	}
 
+	/**
+	 * Cambia la cantidad máxima de amigos que puede colar this.
+	 * 
+	 * @param maxAmigos
+	 *            cantidad de amigos a colar. Debe ser correcta: Mayor o igual a
+	 *            0 y menor de 10.
+	 */
+	public void setMaxAmigos(int maxAmigos) {
+		// TODO Auto-generated method stub
+
+	}
+
 	/**
 	 * 
 	 * @return cantidad de amigos que aún puede colar this.

src/test/java/inf/uva/es/ivagonz/practica4/PersonaConstructorTDDTest.java

@@ -13,7 +13,6 @@ public class PersonaConstructorTDDTest {
 
 	private String nombre;
 	private int id;
-	private int maxAmigos;
 	private ArrayList<Persona> amigos;
 	private ArrayList<Persona> conocidos;
 
@@ -21,7 +20,6 @@ public class PersonaConstructorTDDTest {
 	public void setUp() {
 		nombre = "Pepe";
 		id = 1;
-		maxAmigos = 5;
 		amigos = new ArrayList<Persona>();
 		conocidos = new ArrayList<Persona>();
 	}
@@ -30,18 +28,16 @@ public class PersonaConstructorTDDTest {
 	public void tearDown() {
 		nombre = null;
 		id = 0;
-		maxAmigos = 0;
 		amigos = null;
 		conocidos = null;
 	}
 
 	@Test
 	public void constructorCorrectoTest() {
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 
 		assertEquals(nombre, p.getNombre());
 		assertEquals(id, p.getId());
-		assertEquals(maxAmigos, p.getMaxAmigos());
 		assertEquals(amigos, p.getAmigos());
 		assertEquals(conocidos, p.getConocidos());
 	}
@@ -50,51 +46,48 @@ public class PersonaConstructorTDDTest {
 	public void constructorConNombreCadenaVaciaTest() {
 		nombre = "";
 		@SuppressWarnings("unused")
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 	}
 
 	@Test(expected = IllegalArgumentException.class)
 	public void constructorConNombreNullTest() {
-		maxAmigos = -1;
 		@SuppressWarnings("unused")
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(null, id, amigos, conocidos);
 	}
 
 	@Test(expected = IllegalArgumentException.class)
 	public void constructorConIdNegativoTest() {
 		id = -1;
 		@SuppressWarnings("unused")
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 	}
 
 	@Test(expected = IllegalArgumentException.class)
-	public void constructorConMaxAmigosNegativoTest() {
-		maxAmigos = -1;
+	public void constructorConListaDeAmigosNullTest() {
+		amigos = null;
 		@SuppressWarnings("unused")
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 	}
 
 	@Test(expected = IllegalArgumentException.class)
-	public void constructorConMaxAmigosMayorQueDiezTest() {
-		maxAmigos = 11;
+	public void constructorConListaDeAmigosConElementoNullTest() {
+		amigos.add(null);
 		@SuppressWarnings("unused")
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 	}
 
 	@Test(expected = IllegalArgumentException.class)
-	public void constructorConListaDeAmigosNullTest() {
-		amigos = null;
+	public void constructorConListaDeConocidosNullTest() {
+		conocidos = null;
 		@SuppressWarnings("unused")
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 	}
 
 	@Test(expected = IllegalArgumentException.class)
-	public void constructorConListaDeConocidosNullTest() {
-		conocidos = null;
+	public void constructorConListaDeConocidosConElementoNullTest() {
+		conocidos.add(null);
 		@SuppressWarnings("unused")
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 	}
 
-
-
 }

src/test/java/inf/uva/es/ivagonz/practica4/PersonaGettersTDDTest.java

@@ -12,7 +12,6 @@ public class PersonaGettersTDDTest {
 
 	private String nombre;
 	private int id;
-	private int maxAmigos;
 	private ArrayList<Persona> amigos;
 	private ArrayList<Persona> conocidos;
 
@@ -20,7 +19,6 @@ public class PersonaGettersTDDTest {
 	public void setUp() {
 		nombre = "Pepe";
 		id = 1;
-		maxAmigos = 5;
 		amigos = new ArrayList<Persona>();
 		conocidos = new ArrayList<Persona>();
 	}
@@ -29,46 +27,65 @@ public class PersonaGettersTDDTest {
 	public void tearDown() {
 		nombre = null;
 		id = 0;
-		maxAmigos = 0;
 		amigos = null;
 		conocidos = null;
 	}
 
 	@Test
 	public void getNombreCorrectoTest() {
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		assertEquals(nombre, p.getNombre());
 	}
 
 	@Test
 	public void getIdCorrectoTest() {
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		assertEquals(id, p.getId());
 
 	}
 
 	@Test
 	public void getMaxAmigosCorrectoTest() {
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
-		assertEquals(maxAmigos, p.getMaxAmigos());
+		Persona p = new Persona(nombre, id, amigos, conocidos);
+		p.setMaxAmigos(5);
+		assertEquals(5, p.getMaxAmigos());
 	}
 
 	public void getAmigosPorColarCorrectoTest() {
-		int amigosPorColar = maxAmigos;
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
+		int amigosPorColar = p.getMaxAmigos();
 		assertEquals(amigosPorColar, p.getAmigosPorColar());
 	}
 
 	@Test
 	public void getAmigosCorrectoTest() {
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		assertEquals(amigos, p.getAmigos());
 	}
 
 	@Test
 	public void getConocidosCorrectoTest() {
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		assertEquals(conocidos, p.getConocidos());
 	}
 
+	@Test
+	public void setMaxAmigosCorrectoTest() {
+		Persona p = new Persona(nombre, id, amigos, conocidos);
+		p.setMaxAmigos(2);
+		assertEquals(2, p.getMaxAmigos());
+	}
+
+	@Test(expected = IllegalArgumentException.class)
+	public void setMaxAmigosConNumeroNegativoTest() {
+		Persona p = new Persona(nombre, id, amigos, conocidos);
+		p.setMaxAmigos(-1);
+	}
+
+	@Test(expected = IllegalArgumentException.class)
+	public void setMaxAmigosConNumeroMayorDeDiezTest() {
+		Persona p = new Persona(nombre, id, amigos, conocidos);
+		p.setMaxAmigos(11);
+	}
+
 }

src/test/java/inf/uva/es/ivagonz/practica4/PersonaOperacionesTDDTest.java

@@ -12,7 +12,6 @@ public class PersonaOperacionesTDDTest {
 
 	private String nombre;
 	private int id;
-	private int maxAmigos;
 	private ArrayList<Persona> amigos;
 	private ArrayList<Persona> conocidos;
 
@@ -20,7 +19,6 @@ public class PersonaOperacionesTDDTest {
 	public void setUp() {
 		nombre = "Pepe";
 		id = 1;
-		maxAmigos = 5;
 		amigos = new ArrayList<Persona>();
 		conocidos = new ArrayList<Persona>();
 	}
@@ -29,7 +27,6 @@ public class PersonaOperacionesTDDTest {
 	public void tearDown() {
 		nombre = null;
 		id = 0;
-		maxAmigos = 0;
 		amigos = null;
 		conocidos = null;
 	}
@@ -37,39 +34,39 @@ public class PersonaOperacionesTDDTest {
 	@Test
 	public void addConocidoCorrecto() {
 		fail("Quitar en implementacion");
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
-		Persona p2 = new Persona("Manu", 2, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
+		Persona p2 = new Persona("Manu", 2, amigos, conocidos);
 		p.addConocido(p2);
 		assertEquals(p2.getId(), p.getConocidos().get(p.getConocidos().size() - 1).getId());
 	}
 
 	@Test(expected = IllegalArgumentException.class)
 	public void addConocidoConPersonaNullTest() {
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		p.addConocido(null);
 	}
 
 	@Test(expected = IllegalArgumentException.class)
 	public void addConocidoExistenteTest() {
-		Persona p2 = new Persona("Manu", 2, maxAmigos, amigos, conocidos);
+		Persona p2 = new Persona("Manu", 2, amigos, conocidos);
 		conocidos = new ArrayList<Persona>();
 		conocidos.add(p2);
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		p.addConocido(p2);
 	}
 
 	@Test(expected = IllegalArgumentException.class)
 	public void addConocidoConMismoIdQueThisTest() {
-		Persona p2 = new Persona("Manu", id, maxAmigos, amigos, conocidos);
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p2 = new Persona("Manu", id, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		p.addConocido(p2);
 	}
 
 	@Test
 	public void isConocidoCorrectoTest() {
 		fail("Quitar en implementación");
-		Persona p2 = new Persona("Manu", id, maxAmigos, amigos, conocidos);
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p2 = new Persona("Manu", id, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		p.addConocido(p2);
 		p.isConocido(p2);
 		assertTrue(p.isConocido(p2));
@@ -77,15 +74,15 @@ public class PersonaOperacionesTDDTest {
 
 	@Test(expected = IllegalArgumentException.class)
 	public void isConocidoConConocidoNullTest() {
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		p.isConocido(null);
 	}
 
 	@Test
 	public void addAmigoCorrectoTest() {
 		fail("Quitar en implementación");
-		Persona p2 = new Persona("Manu", id, maxAmigos, amigos, conocidos);
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p2 = new Persona("Manu", id, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		p.addConocido(p2);
 		p.addAmigo(p2);
 		assertEquals(p2.getId(), p.getAmigos().get(p.getAmigos().size() - 1).getId());
@@ -93,32 +90,32 @@ public class PersonaOperacionesTDDTest {
 
 	@Test(expected = IllegalArgumentException.class)
 	public void addAmigoConPersonaNullTest() {
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		p.addAmigo(null);
 	}
 
 	@Test(expected = IllegalArgumentException.class)
 	public void addAmigoExistenteTest() {
-		Persona p2 = new Persona("Manu", 2, maxAmigos, amigos, conocidos);
+		Persona p2 = new Persona("Manu", 2, amigos, conocidos);
 		amigos = new ArrayList<Persona>();
 		amigos.add(p2);
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		p.addConocido(p2);
 		p.addAmigo(p2);
 	}
 
 	@Test(expected = IllegalArgumentException.class)
 	public void addAmigoQueNoEsConocidoTest() {
-		Persona p2 = new Persona("Manu", 2, maxAmigos, amigos, conocidos);
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p2 = new Persona("Manu", 2, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		p.addAmigo(p2);
 	}
 
 	@Test
 	public void isAmigoCorrectoTest() {
 		fail("Quitar en implementación");
-		Persona p2 = new Persona("Manu", id, maxAmigos, amigos, conocidos);
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p2 = new Persona("Manu", id, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		p.addConocido(p2);
 		p.addAmigo(p2);
 		p.isAmigo(p2);
@@ -127,15 +124,15 @@ public class PersonaOperacionesTDDTest {
 
 	@Test(expected = IllegalArgumentException.class)
 	public void isAmigoConAmigoNullTest() {
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		p.isAmigo(null);
 	}
 
 	@Test
 	public void removeAmigoCorrectoTest() {
 		fail("Quitar en implementación");
-		Persona p2 = new Persona("Manu", id, maxAmigos, amigos, conocidos);
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p2 = new Persona("Manu", id, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		p.addConocido(p2);
 		p.addAmigo(p2);
 		p.removeAmigo(p2);
@@ -144,8 +141,8 @@ public class PersonaOperacionesTDDTest {
 
 	@Test(expected = IllegalArgumentException.class)
 	public void removeAmigoConAmigoNullTest() {
-		Persona p2 = new Persona("Manu", id, maxAmigos, amigos, conocidos);
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p2 = new Persona("Manu", id, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		p.addConocido(p2);
 		p.addAmigo(p2);
 		p.removeAmigo(null);
@@ -153,8 +150,8 @@ public class PersonaOperacionesTDDTest {
 
 	@Test(expected = IllegalArgumentException.class)
 	public void removeAmigoConAmigoQueNoExisteTest() {
-		Persona p2 = new Persona("Manu", id, maxAmigos, amigos, conocidos);
-		Persona p = new Persona(nombre, id, maxAmigos, amigos, conocidos);
+		Persona p2 = new Persona("Manu", id, amigos, conocidos);
+		Persona p = new Persona(nombre, id, amigos, conocidos);
 		p.addConocido(p2);
 		p.removeAmigo(p2);
 	}