From 32b6e6f7c649ed41d5b1f697df968b7f7be0eacb Mon Sep 17 00:00:00 2001
From: "Dr. Martin Goik" <goik@hdm-stuttgart.de>
Date: Tue, 3 Jul 2018 08:26:03 +0200
Subject: [PATCH] Javadoc error corrected, partly assertThat() hamcrest support
---
.../mi/sd1/test/aufgabe2/Test_Ersatzteil.java | 108 ++++++++++--------
1 file changed, 63 insertions(+), 45 deletions(-)
diff --git a/Klausuren/Sd1/2017Summer/Solve/src/test/java/de/hdm_stuttgart/mi/sd1/test/aufgabe2/Test_Ersatzteil.java b/Klausuren/Sd1/2017Summer/Solve/src/test/java/de/hdm_stuttgart/mi/sd1/test/aufgabe2/Test_Ersatzteil.java
index 1ec173280..07112c572 100644
--- a/Klausuren/Sd1/2017Summer/Solve/src/test/java/de/hdm_stuttgart/mi/sd1/test/aufgabe2/Test_Ersatzteil.java
+++ b/Klausuren/Sd1/2017Summer/Solve/src/test/java/de/hdm_stuttgart/mi/sd1/test/aufgabe2/Test_Ersatzteil.java
@@ -14,6 +14,12 @@ import de.hdm_stuttgart.mi.sd1.aufgabe2.Ersatzteil;
import de.hdm_stuttgart.mi.sd1.test.helper.ReflectClass;
import de.hdm_stuttgart.mi.sd1.test.helper.ReflectionHelper;
+import static org.hamcrest.CoreMatchers.is;
+import static org.hamcrest.CoreMatchers.equalTo;
+import static org.hamcrest.CoreMatchers.not;
+import static org.hamcrest.core.IsNull.notNullValue;
+import static org.hamcrest.core.IsNull.nullValue;
+
/**
* Teste Ersatzteile.
*
@@ -22,7 +28,7 @@ import de.hdm_stuttgart.mi.sd1.test.helper.ReflectionHelper;
@FixMethodOrder(MethodSorters.NAME_ASCENDING)
public class Test_Ersatzteil extends ExaminationTestDefaults {
- final ReflectionHelper rHelper = new ReflectionHelper();
+ private final ReflectionHelper rHelper = new ReflectionHelper();
/**
* Korrekte toString() Redefinition.
@@ -30,7 +36,9 @@ public class Test_Ersatzteil extends ExaminationTestDefaults {
@Test
@Marking(points = 2)
public void test_010_toString() {
- Assert.assertEquals("Holzbalken, Teilenummer:123", new Ersatzteil(123, "Holzbalken").toString());
+ Assert.assertThat(
+ new Ersatzteil(123, "Holzbalken").toString(),
+ is(equalTo("Holzbalken, Teilenummer:123")));
}
/**
* Korrekte toString() Redefinition.
@@ -38,7 +46,9 @@ public class Test_Ersatzteil extends ExaminationTestDefaults {
@Test
@Marking(points = 1)
public void test_015_toString() {
- Assert.assertEquals("Bezeichnung unbekannt, Teilenummer:123", new Ersatzteil(123, null).toString());
+ Assert.assertThat(
+ new Ersatzteil(123, null).toString(),
+ is(equalTo("Bezeichnung unbekannt, Teilenummer:123")));
}
/**
* Objektgleichheit positiv, gleiche Bezeichnung
@@ -46,11 +56,15 @@ public class Test_Ersatzteil extends ExaminationTestDefaults {
@Test
@Marking(points = 2)
public void test_020_equal() {
- Assert.assertEquals("Keine (geeignete) Ersatzteil.equals() Methode vorhanden! ",
- null,
- rHelper.ersatzteil_equals.getRecursiveErrorMsg());
- Assert.assertEquals(true, new Ersatzteil(23, "Mutter").equals(
- new Ersatzteil(23, "Mutter")));
+
+ Assert.assertThat(
+ "Keine (geeignete) Ersatzteil.equals() Methode vorhanden! ",
+ rHelper.ersatzteil_equals.getRecursiveErrorMsg(),
+ nullValue());
+
+ Assert.assertThat(
+ new Ersatzteil(23, "Mutter"),
+ is(equalTo(new Ersatzteil(23, "Mutter"))));
}
/**
* Objektgleichheit positiv, ungleiche Bezeichnung
@@ -58,11 +72,16 @@ public class Test_Ersatzteil extends ExaminationTestDefaults {
@Test
@Marking(points = 2)
public void test_030_equalBezeichnungAnders() {
- Assert.assertEquals("Keine (geeignete) Ersatzteil.equals() Methode vorhanden! ",
- null,
- rHelper.ersatzteil_equals.getRecursiveErrorMsg());
- Assert.assertEquals(true, new Ersatzteil(23, "Mutter").equals(
- new Ersatzteil(23, "Schraube")));
+
+ Assert.assertThat(
+ "Keine (geeignete) Ersatzteil.equals() Methode vorhanden! ",
+ rHelper.ersatzteil_equals.getRecursiveErrorMsg(),
+ nullValue()
+ );
+
+ Assert.assertThat(
+ new Ersatzteil(23, "Schraube"),
+ is(equalTo(new Ersatzteil(23, "Mutter"))));
}
/**
* Objektgleichheit negativ, gleiche Bezeichnung
@@ -70,24 +89,24 @@ public class Test_Ersatzteil extends ExaminationTestDefaults {
@Test
@Marking(points = 1)
public void test_040_verschieden() {
-
- Assert.assertNull(rHelper.ersatzteil_equals.getRecursiveErrorMsg());
- Assert.assertNull(rHelper.ersatzteil_equals.assertIsDefinedIn(Ersatzteil.class));
+ Assert.assertThat(rHelper.ersatzteil_equals.getRecursiveErrorMsg(), nullValue());
+ Assert.assertThat(rHelper.ersatzteil_equals.assertIsDefinedIn(Ersatzteil.class),nullValue());
+
+ Assert.assertThat(new Ersatzteil(24, "Mutter"),
+ not(is(equalTo(new Ersatzteil(23, "Mutter")))));
- Assert.assertEquals(false, new Ersatzteil(23, "Mutter").equals(
- new Ersatzteil(24, "Mutter")));
}
/**
* Objektgleichheit negativ, ungleiche Bezeichnung
*/
@Test
@Marking(points = 2)
- public void test_050_verschiedenBezeichnung() {
- Assert.assertNull(rHelper.ersatzteil_equals.getRecursiveErrorMsg());
- Assert.assertNull(rHelper.ersatzteil_equals.assertIsDefinedIn(Ersatzteil.class));
+ public void test_050_verschiedenBezeichnung() {
- Assert.assertEquals(false, new Ersatzteil(23, "Mutter").equals(
- new Ersatzteil(24, "Schraube")));
+ Assert.assertThat(rHelper.ersatzteil_equals.getRecursiveErrorMsg(), nullValue());
+ Assert.assertThat(rHelper.ersatzteil_equals.assertIsDefinedIn(Ersatzteil.class), nullValue());
+ Assert.assertThat(new Ersatzteil(24, "Schraube"),
+ not(is(equalTo(new Ersatzteil(23, "Mutter")))));
}
/**
* Objektgleichheit negativ, Vergleich mit null
@@ -95,10 +114,9 @@ public class Test_Ersatzteil extends ExaminationTestDefaults {
@Test
@Marking(points = 2)
public void test_060_verschiedenNull() {
- Assert.assertNull(rHelper.ersatzteil_equals.getRecursiveErrorMsg());
- Assert.assertNull(rHelper.ersatzteil_equals.assertIsDefinedIn(Ersatzteil.class));
-
- Assert.assertEquals(false, new Ersatzteil(23, "Mutter").equals(null));
+ Assert.assertThat(rHelper.ersatzteil_equals.getRecursiveErrorMsg(), nullValue());
+ Assert.assertThat(rHelper.ersatzteil_equals.assertIsDefinedIn(Ersatzteil.class), nullValue());
+ Assert.assertThat(null, not(equalTo(new Ersatzteil(23, "Mutter"))));
}
/**
* Objektgleichheit negativ, Vergleich mit String
@@ -106,10 +124,10 @@ public class Test_Ersatzteil extends ExaminationTestDefaults {
@Test
@Marking(points = 1)
public void test_070_verschiedenString() {
- Assert.assertNull(rHelper.ersatzteil_equals.getRecursiveErrorMsg());
- Assert.assertNull(rHelper.ersatzteil_equals.assertIsDefinedIn(Ersatzteil.class));
- Assert.assertEquals(false, new Ersatzteil(23, "Mutter").equals("irgendwas"));
+ Assert.assertThat(rHelper.ersatzteil_equals.getRecursiveErrorMsg(), nullValue());
+ Assert.assertThat(rHelper.ersatzteil_equals.assertIsDefinedIn(Ersatzteil.class), nullValue());
+ Assert.assertThat("irgendwas", not(equalTo(new Ersatzteil(23, "Mutter"))));
}
/**
@@ -118,13 +136,13 @@ public class Test_Ersatzteil extends ExaminationTestDefaults {
@Test
@Marking(points = 1)
public void test_075_attrib_ersatzteil_private() {
-
+
Assert.assertTrue("Die Klassse 'Ersatzteil' hat kein Attribut", 0 < Ersatzteil.class.getDeclaredFields().length);
final String nonPrivateAttribs = ReflectClass.getNonPrivateAttributes(Ersatzteil.class);
- Assert.assertEquals("Folgende Attribute in der Klasse 'Ersatzteil' sind nicht 'private': " + nonPrivateAttribs + ". ",
- null,
- nonPrivateAttribs);
+ Assert.assertNull(
+ "Folgende Attribute in der Klasse 'Ersatzteil' sind nicht 'private': " + nonPrivateAttribs + ". ",
+ nonPrivateAttribs);
}
/**
@@ -133,17 +151,17 @@ public class Test_Ersatzteil extends ExaminationTestDefaults {
@Test
@Marking(points = 1)
public void test_080_classReifen() {
- Assert.assertEquals(null, rHelper.reifenKlasse.msg);
+ Assert.assertNull(rHelper.reifenKlasse.msg);
}
- /** modifier & method.getModifiers();
+ /** modifier and method.getModifiers();
* Konstruktor Definition public Reifen (int, int){...}
*/
@Test
@Marking(points = 1)
public void test_200_reifenKonstruktor() {
- Assert.assertEquals(null, rHelper.reifen_constructor_int_int.getRecursiveMsg());
+ Assert.assertNull(rHelper.reifen_constructor_int_int.getRecursiveMsg());
}
/**
@@ -164,7 +182,7 @@ public class Test_Ersatzteil extends ExaminationTestDefaults {
@Test
@Marking(points = 1)
public void test_220_Reifen_toString() {
- Assert.assertEquals(null, rHelper.reifen_toString.getRecursiveErrorMsg());
+ Assert.assertNull(rHelper.reifen_toString.getRecursiveErrorMsg());
Assert.assertNull(rHelper.reifen_toString.assertIsDefinedIn("de.hdm_stuttgart.mi.sd1.aufgabe2.Reifen"));
Assert.assertNull(rHelper.reifen_toString.assertModifierIsPresent(Modifier.PUBLIC));
@@ -172,17 +190,17 @@ public class Test_Ersatzteil extends ExaminationTestDefaults {
}
/**
- * Erzeugung Instanz Reifen(final int number) und pruefung toString()
+ * Erzeugung Instanz Reifen(final int number) ,Pruefung toString()
*/
@Test
@Marking(points = 2)
public void test_400_konstruktorReifen() {
- Assert.assertEquals(null, rHelper.reifenKlasse.msg);
- Assert.assertEquals(null, rHelper.reifen_constructor_int_int.getRecursiveMsg());
+ Assert.assertNull(rHelper.reifenKlasse.msg);
+ Assert.assertNull(rHelper.reifen_constructor_int_int.getRecursiveMsg());
- Object reifen = null;
+ Object reifen;
try {
- reifen = rHelper.reifen_constructor_int_int.constructor.newInstance(new Object[] {1132, 160});
+ reifen = rHelper.reifen_constructor_int_int.constructor.newInstance(1132, 160);
} catch (InstantiationException | IllegalAccessException | IllegalArgumentException
| InvocationTargetException e) {
Assert.fail("Konnte 'new Reifen(1132, 160)' nicht aufrufen.");
@@ -190,10 +208,10 @@ public class Test_Ersatzteil extends ExaminationTestDefaults {
}
Assert.assertTrue("Objekt ist kein Ersatzteil", reifen instanceof Ersatzteil);
try {
- Assert.assertEquals("Reifen, Teilenummer:1132, Maximalgeschwindigkeit:160 Kmh", rHelper.reifen_toString.method.invoke(reifen, new Object[] {}));
+ Assert.assertEquals("Reifen, Teilenummer:1132, Maximalgeschwindigkeit:160 Kmh",
+ rHelper.reifen_toString.method.invoke(reifen));
} catch (IllegalAccessException | IllegalArgumentException | InvocationTargetException e) {
Assert.fail("Konnte 'toString()' für 'new Reifen(1132, 160)' nicht aufrufen.");
- return;
}
}
}
\ No newline at end of file
--
GitLab