Merge pull request 'Test der Magierkarte und Narrenkarte angepasst.' (#37) from kts.desktop into main

Reviewed-on: 2211945/WIZARD_PR2_DOP#37
Reviewed-by: Philipp Kotte <2211945@stud.hs-mannheim.de>

Domain/Karten/Magierkarte.java

@@ -6,43 +6,25 @@ written on: 05 / 10 / 2023 at: 23:41
 */
 package Domain.Karten;
 
-import Domain.Enums.Kartenfarbe;
-
 public class Magierkarte extends Karte {
 
 	// Statische Konstanten
-
+	private static String TRUMPF_NAME = "Zauberer";
 	// Statische Attribute
 
 	// Attribute des Objekts
-	private Kartenfarbe kartenfarbe;
 	private String bezeichnung;
 	// Konstruktoren
 
 	public Magierkarte() {
 		super();
-
+		setBezeichnung(TRUMPF_NAME);
-	}
-
-	public Magierkarte(Kartenfarbe kartenfarbe, String bezeichnung) {
-		super();
-		setKartenfarbe(kartenfarbe);
-		setBezeichnung(bezeichnung);
-
 	}
 
 	// Statische Methoden
 
 	// Getter und Setter
 
-	public void setKartenfarbe(Kartenfarbe kartenfarbe) {
-		this.kartenfarbe = kartenfarbe;
-	}
-
-	public Kartenfarbe getKartenfarbe() {
-		return this.kartenfarbe;
-	}
-
 	public void setBezeichnung(String bezeichnung) {
 		this.bezeichnung = bezeichnung;
 	}
@@ -54,7 +36,7 @@ public class Magierkarte extends Karte {
 	// @Overrides
 	@Override
 	public String toString() {
-		return this.getBezeichnung() + ": " + this.getKartenfarbe().name() + " (" + this.getKartenfarbe().toString() +").";
+		return this.getBezeichnung();
 	}
 	// Public Methoden
 

Domain/Karten/Narrenkarte.java

@@ -6,41 +6,25 @@ written on: 05 / 10 / 2023 at: 23:41
 */
 package Domain.Karten;
 
-import Domain.Enums.Kartenfarbe;
-
 public class Narrenkarte extends Karte {
 
 	// Statische Konstanten
-
+	private static String TRUMPF_NAME = "Narr";
 	// Statische Attribute
 
 	// Attribute des Objekts
-	private Kartenfarbe kartenfarbe;
 	private String bezeichnung;
 	// Konstruktoren
 
 	public Narrenkarte() {
 		super();
+		setBezeichnung(TRUMPF_NAME);
 	}
 
-	public Narrenkarte(Kartenfarbe farbe, String bezeichnung) {
-		super();
-		setKartenfarbe(farbe);
-		setBezeichnung(bezeichnung);
-
-	}
 	// Statische Methoden
 
 	// Getter und Setter
 
-	public void setKartenfarbe(Kartenfarbe farbe) {
-		this.kartenfarbe = farbe;
-	}
-
-	public Kartenfarbe getKartenfarbe() {
-		return this.kartenfarbe;
-	}
-
 	public void setBezeichnung(String bezeichnung) {
 		this.bezeichnung = bezeichnung;
 	}
@@ -52,7 +36,7 @@ public class Narrenkarte extends Karte {
 	// @Overrides
 	@Override
 	public String toString() {
-		return this.getBezeichnung() + ": " + this.getKartenfarbe().name() + " (" + this.getKartenfarbe().toString() + ").";
+		return this.getBezeichnung();
 	}
 	// Public Methoden
 

Test/Domain/Karten/MagierkarteTest.java

@@ -7,16 +7,6 @@ written on: 10 / 10 / 2023 at: 21:18
 package Test.Domain.Karten;
 
 import static org.junit.Assert.assertEquals; 
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertThrows;
-
-import java.util.ArrayList; 
-
-import org.junit.After; 
-import org.junit.AfterClass; 
-import org.junit.Before; 
-import org.junit.BeforeClass; 
-import org.junit.Ignore; 
 import org.junit.Test; 
 import Domain.Enums.Kartenfarbe;
 import Domain.Karten.Magierkarte;
@@ -33,23 +23,21 @@ public class MagierkarteTest {
 
 	@Test
 	public void vierMagier() {
-		Magierkarte m1 = new Magierkarte(Kartenfarbe.MENSCH, "Zauberer");
+		Magierkarte m1 = new Magierkarte();
-		Magierkarte m2 = new Magierkarte(Kartenfarbe.ELF, "Zauberer");
+		Magierkarte m2 = new Magierkarte();
-		Magierkarte m3 = new Magierkarte(Kartenfarbe.RIESE, "Zauberer");
+		Magierkarte m3 = new Magierkarte();
-		Magierkarte m4 = new Magierkarte(Kartenfarbe.ZWERG, "Zauberer");
+		Magierkarte m4 = new Magierkarte();
+		
 		
-		assertEquals(Kartenfarbe.MENSCH, m1.getKartenfarbe());
 		assertEquals("Zauberer", m1.getBezeichnung());
-		assertEquals("BLAU", Kartenfarbe.MENSCH.getFarbe());
+		
-		assertEquals(Kartenfarbe.ELF, m2.getKartenfarbe());
+		
 		assertEquals("Zauberer", m2.getBezeichnung());
-		assertEquals("GRÜN", Kartenfarbe.ELF.getFarbe());
+		
-		assertEquals(Kartenfarbe.RIESE, m3.getKartenfarbe());
 		assertEquals("Zauberer", m3.getBezeichnung());
-		assertEquals("GELB", Kartenfarbe.RIESE.getFarbe());
+		
-		assertEquals(Kartenfarbe.ZWERG, m4.getKartenfarbe());
 		assertEquals("Zauberer", m4.getBezeichnung());
-		assertEquals("ROT", Kartenfarbe.ZWERG.getFarbe());
+		
 		
 		assertEquals("Zauberer: MENSCH (BLAU).", m1.toString());
 		

Test/Domain/Karten/NarrenkarteTest.java

@@ -7,35 +7,23 @@ written on: 10 / 10 / 2023 at: 21:18
 package Test.Domain.Karten;
 
 import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertThrows;
-
 import org.junit.Test;
 
-import Domain.Enums.Kartenfarbe;
 import Domain.Karten.Narrenkarte;
 
 public class NarrenkarteTest {
 
 	@Test
 	public void vierNarren() {
-		Narrenkarte n1 = new Narrenkarte(Kartenfarbe.MENSCH, "Narr");
+		Narrenkarte n1 = new Narrenkarte();
-		Narrenkarte n2 = new Narrenkarte(Kartenfarbe.ELF, "Narr");
+		Narrenkarte n2 = new Narrenkarte();
-		Narrenkarte n3 = new Narrenkarte(Kartenfarbe.RIESE, "Narr");
+		Narrenkarte n3 = new Narrenkarte();
-		Narrenkarte n4 = new Narrenkarte(Kartenfarbe.ZWERG, "Narr");
+		Narrenkarte n4 = new Narrenkarte();
-		
+
-		assertEquals(n1.getKartenfarbe(), Kartenfarbe.MENSCH);
-		assertEquals(n2.getKartenfarbe(), Kartenfarbe.ELF);
-		assertEquals(n3.getKartenfarbe(), Kartenfarbe.RIESE);
-		assertEquals(n4.getKartenfarbe(), Kartenfarbe.ZWERG);
-		
 		assertEquals(n1.getBezeichnung(), "Narr");
 		assertEquals(n2.getBezeichnung(), "Narr");
 		assertEquals(n3.getBezeichnung(), "Narr");
 		assertEquals(n4.getBezeichnung(), "Narr");
-		
+
-		assertEquals(Kartenfarbe.MENSCH.getFarbe(), "BLAU");
-		assertEquals(Kartenfarbe.ELF.getFarbe(), "GRÜN");
-		assertEquals(Kartenfarbe.RIESE.getFarbe(), "GELB");
-		assertEquals(Kartenfarbe.ZWERG.getFarbe(), "ROT");
 	}
 }

Test/Domain/Karten/ZahlenkarteTest.java

@@ -10,54 +10,56 @@ import static org.junit.Assert.assertEquals;
 
 import java.util.HashMap;
 
-import org.junit.jupiter.api.Test;
+import org.junit.Test;
 
 import Domain.Enums.Kartenfarbe;
 import Domain.Karten.Zahlenkarte;
 
 public class ZahlenkarteTest {
-	
+
 	/**
-	 * Die Objekterzeugung der HashMap und der Zahlenkarten 
+	 * Die Objekterzeugung der HashMap und der Zahlenkarten
 	 * kann auch in ein @BeforeAll eingetragen werden.
 	 */
 
 	/**
-	 * Der @Test erzeugt Zahlenkarten von 1 bis 13. prüft aber 
+	 * Der @Test erzeugt Zahlenkarten von 1 bis 13. prüft aber
-	 * nur zwei Positionen in der HashMap, um die 
+	 * nur zwei Positionen in der HashMap, um die
 	 * Objektdaten zu validieren.
 	 */
 	@Test
 	public void dreizehnZahlenkartenMENSCH() {
-		
+
 		HashMap<Integer, Zahlenkarte> zm = new HashMap<Integer, Zahlenkarte>();
 		Zahlenkarte zahlenkarte = null;
 		for (int i = 1; i <= 13; i++) {
 			zahlenkarte = new Zahlenkarte(Kartenfarbe.MENSCH, i);
-			zm.put (i, zahlenkarte);	
+			zm.put(i, zahlenkarte);
 		}
-		
+
 		assertEquals(zm.get(7).getKartenfarbe(), Kartenfarbe.MENSCH);
 		assertEquals(zm.get(7).getKartenNummer(), 7);
-		
+
 		assertEquals(zm.get(4).getKartenfarbe(), Kartenfarbe.MENSCH);
 		assertEquals(zm.get(4).getKartenNummer(), 4);
 	}
+
 	@Test
 	public void dreizehnZahlenkartenElf() {
 		HashMap<Integer, Zahlenkarte> ze = new HashMap<Integer, Zahlenkarte>();
 		Zahlenkarte zahlenkarte = null;
 		for (int i = 1; i <= 13; i++) {
 			zahlenkarte = new Zahlenkarte(Kartenfarbe.ELF, i);
-			ze.put (i, zahlenkarte);	
+			ze.put(i, zahlenkarte);
 		}
-		
+
 		assertEquals(ze.get(7).getKartenfarbe(), Kartenfarbe.ELF);
 		assertEquals(ze.get(7).getKartenNummer(), 7);
-		
+
 		assertEquals(ze.get(4).getKartenfarbe(), Kartenfarbe.ELF);
 		assertEquals(ze.get(4).getKartenNummer(), 4);
 	}
+
 	@Test
 	public void dreizehnZahlenkartenZWERG() {
 		HashMap<Integer, Zahlenkarte> zz = new HashMap<Integer, Zahlenkarte>();
@@ -66,26 +68,27 @@ public class ZahlenkarteTest {
 			zahlenkarte = new Zahlenkarte(Kartenfarbe.ZWERG, i);
 			zz.put(i, zahlenkarte);
 		}
-		
+
 		assertEquals(zz.get(7).getKartenfarbe(), Kartenfarbe.ZWERG);
 		assertEquals(zz.get(7).getKartenNummer(), 7);
-		
+
 		assertEquals(zz.get(4).getKartenfarbe(), Kartenfarbe.ZWERG);
 		assertEquals(zz.get(4).getKartenNummer(), 4);
-		
+
 	}
+
 	@Test
 	public void dreizehnZahlenkartenRIESE() {
 		HashMap<Integer, Zahlenkarte> zr = new HashMap<Integer, Zahlenkarte>();
 		Zahlenkarte zahlenkarte = null;
 		for (int i = 1; i <= 13; i++) {
 			zahlenkarte = new Zahlenkarte(Kartenfarbe.RIESE, i);
-			zr.put(i,  zahlenkarte);
+			zr.put(i, zahlenkarte);
 		}
-		
+
 		assertEquals(zr.get(9).getKartenfarbe(), Kartenfarbe.RIESE);
 		assertEquals(zr.get(9).getKartenNummer(), 9);
-		
+
 		assertEquals(zr.get(2).getKartenfarbe(), Kartenfarbe.RIESE);
 		assertEquals(zr.get(2).getKartenNummer(), 2);
 	}