Changeset 8880 in josm for trunk/test/unit/org/openstreetmap

Timestamp:

2015-10-15T00:11:11+02:00 (9 years ago)

Author:

Don-vip

Message:

checkstyle + remove call to org.hamcrest.CoreMatchers.is (generating a deprecation warning)

File:

• trunk/test/unit/org/openstreetmap/josm/gui/conflict/tags/RelationMemberConflictResolverModelTest.java (modified) (5 diffs)

trunk/test/unit/org/openstreetmap/josm/gui/conflict/tags/RelationMemberConflictResolverModelTest.java

@@ +1 @@
+// License: GPL. For details, see LICENSE file.
 package org.openstreetmap.josm.gui.conflict.tags;
 
-import static org.hamcrest.CoreMatchers.is;
+import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertThat;
 import static org.junit.Assert.assertTrue;
 
@@ -16 +16 @@
 import org.openstreetmap.josm.data.osm.DataSet;
 import org.openstreetmap.josm.data.osm.Node;
-import org.openstreetmap.josm.data.osm.OsmPrimitive;
 import org.openstreetmap.josm.data.osm.Relation;
 import org.openstreetmap.josm.data.osm.RelationMember;
@@ -23 +22 @@
 public class RelationMemberConflictResolverModelTest {
 
+    /**
+     * Setup test.
+     */
     @BeforeClass
     public static void init() {
@@ -62 +64 @@
         model.prepareDefaultRelationDecisions();
         assertTrue(model.isResolvedCompletely());
-        assertThat(model.getDecision(0).getDecision(), is(RelationMemberConflictDecisionType.KEEP));
-        assertThat(model.getDecision(0).getOriginalPrimitive(), is((OsmPrimitive) ways.get(0)));
-        assertThat(model.getDecision(0).getRole(), is("foo"));
-        assertThat(model.getDecision(1).getDecision(), is(RelationMemberConflictDecisionType.REMOVE));
-        assertThat(model.getDecision(1).getOriginalPrimitive(), is((OsmPrimitive) ways.get(1)));
+        assertEquals(RelationMemberConflictDecisionType.KEEP, model.getDecision(0).getDecision());
+        assertEquals(ways.get(0), model.getDecision(0).getOriginalPrimitive());
+        assertEquals("foo", model.getDecision(0).getRole());
+        assertEquals(RelationMemberConflictDecisionType.REMOVE, model.getDecision(1).getDecision());
+        assertEquals(ways.get(1), model.getDecision(1).getOriginalPrimitive());
     }
 
@@ -109 +111 @@
         model.prepareDefaultRelationDecisions();
         assertTrue(model.isResolvedCompletely());
-        assertThat(model.getDecision(0).getDecision(), is(RelationMemberConflictDecisionType.KEEP));
-        assertThat(model.getDecision(0).getOriginalPrimitive(), is((OsmPrimitive) ways.get(0)));
-        assertThat(model.getDecision(0).getRole(), is("foo"));
-        assertThat(model.getDecision(1).getDecision(), is(RelationMemberConflictDecisionType.KEEP));
-        assertThat(model.getDecision(1).getOriginalPrimitive(), is((OsmPrimitive) ways.get(0)));
-        assertThat(model.getDecision(1).getRole(), is("bar"));
-        assertThat(model.getDecision(2).getDecision(), is(RelationMemberConflictDecisionType.REMOVE));
-        assertThat(model.getDecision(3).getDecision(), is(RelationMemberConflictDecisionType.REMOVE));
+        assertEquals(RelationMemberConflictDecisionType.KEEP, model.getDecision(0).getDecision());
+        assertEquals(ways.get(0), model.getDecision(0).getOriginalPrimitive());
+        assertEquals("foo", model.getDecision(0).getRole());
+        assertEquals(RelationMemberConflictDecisionType.KEEP, model.getDecision(1).getDecision());
+        assertEquals(ways.get(0), model.getDecision(1).getOriginalPrimitive());
+        assertEquals("bar", model.getDecision(1).getRole());
+        assertEquals(RelationMemberConflictDecisionType.REMOVE, model.getDecision(2).getDecision());
+        assertEquals(RelationMemberConflictDecisionType.REMOVE, model.getDecision(3).getDecision());
     }
 }