summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorCarsten Drossel2008-08-18 09:10:45 (EDT)
committerCarsten Drossel2008-08-18 09:10:45 (EDT)
commit30bd5abf5e674ad9cbc5f0763cbf557b26d84dae (patch)
treedc07198b97a202ad7ccc80396f04cb052004ecba
parent580e6d663daa95fe7f8a527baca2bcfa61a72ee7 (diff)
downloadorg.eclipse.riena-30bd5abf5e674ad9cbc5f0763cbf557b26d84dae.zip
org.eclipse.riena-30bd5abf5e674ad9cbc5f0763cbf557b26d84dae.tar.gz
org.eclipse.riena-30bd5abf5e674ad9cbc5f0763cbf557b26d84dae.tar.bz2
refactored test to use the full variety of assert-methods
-rw-r--r--org.eclipse.riena.tests/src/org/eclipse/riena/objecttransaction/list/ObjectTransactionListTest.java36
-rw-r--r--org.eclipse.riena.tests/src/org/eclipse/riena/objecttransaction/simple/ObjectTransactionVariousSimpleTest.java24
2 files changed, 30 insertions, 30 deletions
diff --git a/org.eclipse.riena.tests/src/org/eclipse/riena/objecttransaction/list/ObjectTransactionListTest.java b/org.eclipse.riena.tests/src/org/eclipse/riena/objecttransaction/list/ObjectTransactionListTest.java
index 9b3e751..5a911f4 100644
--- a/org.eclipse.riena.tests/src/org/eclipse/riena/objecttransaction/list/ObjectTransactionListTest.java
+++ b/org.eclipse.riena.tests/src/org/eclipse/riena/objecttransaction/list/ObjectTransactionListTest.java
@@ -872,8 +872,8 @@ public class ObjectTransactionListTest extends RienaTestCase {
Kunde kunde = new Kunde("4711");
kunde.setVorname("john");
kunde.setNachname("Miller");
- assertTrue("vorname=john", kunde.getVorname().equals("john"));
- assertTrue("nachname=Miller", kunde.getNachname().equals("Miller"));
+ assertEquals("vorname=john", "john", kunde.getVorname());
+ assertEquals("nachname=Miller", "Miller", kunde.getNachname());
Vertrag v1 = new Vertrag("0815");
v1.setVertragsBeschreibung("mein erster Vertrag");
@@ -882,19 +882,19 @@ public class ObjectTransactionListTest extends RienaTestCase {
Vertrag v2 = new Vertrag("0816");
v2.setVertragsBeschreibung("noch ein Vertrag");
kunde.addVertrag(v2);
- assertTrue("anzahl verträge bei kunde ist falsch", kunde.listVertrag().length == 2);
+ assertEquals("anzahl verträge bei kunde ist falsch", 2, kunde.listVertrag().length);
// making changes to the object
objectTransaction.setCleanModus(false);
kunde.setVorname("jane");
kunde.setNachname("Stewart");
- assertTrue("reihenfolge: erste Vertrag ist nicht 0815", kunde.listVertrag()[0] == v1);
- assertTrue("reihenfolge: zweiter Vertrag ist nicht 0816", kunde.listVertrag()[1] == v2);
+ assertSame("reihenfolge: erste Vertrag ist nicht 0815", kunde.listVertrag()[0], v1);
+ assertSame("reihenfolge: zweiter Vertrag ist nicht 0816", kunde.listVertrag()[1], v2);
kunde.removeVertrag("0815");
- assertTrue("anzahl verträge bei kunde ist falsch", kunde.listVertrag().length == 1);
+ assertEquals("anzahl verträge bei kunde ist falsch", 1, kunde.listVertrag().length);
- assertTrue("reihenfolge: erste Vertrag ist nicht 0816", kunde.listVertrag()[0] == v2);
+ assertSame("reihenfolge: erste Vertrag ist nicht 0816", kunde.listVertrag()[0], v2);
// extracting recorded changes
IObjectTransactionExtract extract = objectTransaction.exportExtract();
@@ -908,8 +908,8 @@ public class ObjectTransactionListTest extends RienaTestCase {
kunde2.setVorname("john");
kunde2.setNachname("Miller");
- assertTrue("vorname=john", kunde2.getVorname().equals("john"));
- assertTrue("nachname=Miller", kunde2.getNachname().equals("Miller"));
+ assertEquals("vorname=john", "john", kunde2.getVorname());
+ assertEquals("nachname=Miller", "Miller", kunde2.getNachname());
v1 = new Vertrag("0815");
v1.setVertragsBeschreibung("mein erster Vertrag");
@@ -918,21 +918,21 @@ public class ObjectTransactionListTest extends RienaTestCase {
v2 = new Vertrag("0816");
v2.setVertragsBeschreibung("noch ein Vertrag");
kunde2.addVertrag(v2);
- assertTrue("anzahl verträge bei kunde ist falsch", kunde2.listVertrag().length == 2);
- assertTrue("reihenfolge: erste Vertrag ist nicht 0815", kunde2.listVertrag()[0] == v1);
- assertTrue("reihenfolge: zweiter Vertrag ist nicht 0816", kunde2.listVertrag()[1] == v2);
+ assertEquals("anzahl verträge bei kunde ist falsch", 2, kunde2.listVertrag().length);
+ assertSame("reihenfolge: erste Vertrag ist nicht 0815", kunde2.listVertrag()[0], v1);
+ assertSame("reihenfolge: zweiter Vertrag ist nicht 0816", kunde2.listVertrag()[1], v2);
// apply the changes from the other objecttransaction's extract
objectTransaction2.setCleanModus(false);
objectTransaction2.importExtract(extract);
// ---> properties have changed
- assertTrue("vorname=john", kunde2.getVorname().equals("jane"));
- assertTrue("nachname=Miller", kunde2.getNachname().equals("Stewart"));
- assertTrue("vertrag 0815 must return null", kunde2.getVertrag("0815") == null);
- assertTrue("anzahl verträge bei kunde ist falsch", kunde2.listVertrag().length == 1);
- assertTrue("reihenfolge: erste Vertrag ist nicht 0816", kunde2.listVertrag()[0] == v2);
+ assertEquals("vorname=john", "jane", kunde2.getVorname());
+ assertEquals("nachname=Miller", "Stewart", kunde2.getNachname());
+ assertNull("vertrag 0815 must return null", kunde2.getVertrag("0815"));
+ assertEquals("anzahl verträge bei kunde ist falsch", 1, kunde2.listVertrag().length);
+ assertSame("reihenfolge: erste Vertrag ist nicht 0816", kunde2.listVertrag()[0], v2);
kunde.removeVertrag("0816");
- assertTrue("anzahl verträge bei kunde ist falsch", kunde2.listVertrag().length == 0);
+ assertEquals("anzahl verträge bei kunde ist falsch", 0, kunde2.listVertrag().length);
}
/**
diff --git a/org.eclipse.riena.tests/src/org/eclipse/riena/objecttransaction/simple/ObjectTransactionVariousSimpleTest.java b/org.eclipse.riena.tests/src/org/eclipse/riena/objecttransaction/simple/ObjectTransactionVariousSimpleTest.java
index fcbabde..bef26f6 100644
--- a/org.eclipse.riena.tests/src/org/eclipse/riena/objecttransaction/simple/ObjectTransactionVariousSimpleTest.java
+++ b/org.eclipse.riena.tests/src/org/eclipse/riena/objecttransaction/simple/ObjectTransactionVariousSimpleTest.java
@@ -241,11 +241,11 @@ public class ObjectTransactionVariousSimpleTest extends RienaTestCase {
IObjectTransactionExtract extract = objectTransaction.exportOnlyModifedObjectsToExtract();
TransactionDelta[] deltas = extract.getDeltas();
- assertTrue("should be only two transaction delta", deltas.length == 2);
- assertTrue("single delta should reference v2", deltas[0].getObjectId() == v2.getObjectId());
- assertTrue("delta status must be modified", deltas[0].getState().equals(State.CREATED));
- assertTrue("single delta should reference kunde", deltas[1].getObjectId() == kunde.getObjectId());
- assertTrue("delta status must be modified", deltas[1].getState().equals(State.MODIFIED));
+ assertEquals("should be only two transaction delta", 2, deltas.length);
+ assertSame("single delta should reference v2", deltas[0].getObjectId(), v2.getObjectId());
+ assertEquals("delta status must be modified", State.CREATED, deltas[0].getState());
+ assertSame("single delta should reference kunde", deltas[1].getObjectId(), kunde.getObjectId());
+ assertEquals("delta status must be modified", State.MODIFIED, deltas[1].getState());
}
/**
@@ -270,13 +270,13 @@ public class ObjectTransactionVariousSimpleTest extends RienaTestCase {
IObjectTransactionExtract extract = objectTransaction.exportOnlyModifedObjectsToExtract();
TransactionDelta[] deltas = extract.getDeltas();
- assertTrue("should be only three transaction delta", deltas.length == 3);
- assertTrue("single delta should reference v1", deltas[0].getObjectId() == v2.getObjectId());
- assertTrue("delta status must be modified", deltas[0].getState().equals(State.CREATED));
- assertTrue("single delta should reference v2", deltas[1].getObjectId() == v1.getObjectId());
- assertTrue("delta status must be modified", deltas[1].getState().equals(State.CREATED));
- assertTrue("single delta should reference kunde", deltas[2].getObjectId() == kunde.getObjectId());
- assertTrue("delta status must be modified", deltas[2].getState().equals(State.MODIFIED));
+ assertEquals("should be only three transaction delta", 3, deltas.length);
+ assertSame("single delta should reference v1", deltas[0].getObjectId(), v2.getObjectId());
+ assertEquals("delta status must be modified", State.CREATED, deltas[0].getState());
+ assertSame("single delta should reference v2", deltas[1].getObjectId(), v1.getObjectId());
+ assertEquals("delta status must be modified", State.CREATED, deltas[1].getState());
+ assertSame("single delta should reference kunde", deltas[2].getObjectId(), kunde.getObjectId());
+ assertEquals("delta status must be modified", State.MODIFIED, deltas[2].getState());
}
/**