Merge pull request #609 from exercism/clock-assertions

clock: prefer assert(Not)Equals to assert(True|False)

exercises/clock/HINT.md

+In order to satisfy the requirement that two clocks are considered equal just when they are set to the same time, you will need to override the [`equals`](https://docs.oracle.com/javase/8/docs/api/java/lang/Object.html#equals(java.lang.Object)) and [`hashcode`](https://docs.oracle.com/javase/8/docs/api/java/lang/Object.html#hashCode) methods in your `Clock` class.
+ 
+For more information on how to override these methods, see [this JavaWorld article](https://web.archive.org/web/20170528222153/http://www.javaworld.com/article/2072762/java-app-dev/object-equality.html).

exercises/clock/src/test/java/ClockEqualTest.java

 import org.junit.Ignore;
 import org.junit.Test;
 
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.assertNotEquals;
+import static org.junit.Assert.assertEquals;
 
 public class ClockEqualTest {
 
     @Ignore("Remove to run test")
     @Test
     public void clocksWithSameTimeAreEqual() {
-        assertTrue(new Clock(15, 37).equals(new Clock(15, 37)));
+        assertEquals(new Clock(15, 37), new Clock(15, 37));
     }
 
     @Ignore("Remove to run test")
     @Test
     public void clocksAMinuteApartAreNotEqual() {
-        assertFalse(new Clock(15, 36).equals(new Clock(15, 37)));
+        assertNotEquals(new Clock(15, 36), new Clock(15, 37));
     }
 
     @Ignore("Remove to run test")
     @Test
     public void clocksAnHourApartAreNotEqual() {
-        assertFalse(new Clock(14, 37).equals(new Clock(15, 37)));
+        assertNotEquals(new Clock(14, 37), new Clock(15, 37));
     }
 
     @Ignore("Remove to run test")
     @Test
     public void clocksWithHourOverflow() {
-        assertTrue(new Clock(10, 37).equals(new Clock(34, 37)));
+        assertEquals(new Clock(10, 37), new Clock(34, 37));
     }
 
     @Ignore("Remove to run test")
     @Test
     public void clocksWithHourOverflowBySeveralDays() {
-        assertTrue(new Clock(3, 11).equals(new Clock(99, 11)));
+        assertEquals(new Clock(3, 11), new Clock(99, 11));
     }
 
     @Ignore("Remove to run test")
     @Test
     public void clocksWithNegateHour() {
-        assertTrue(new Clock(22, 40).equals(new Clock(-2, 40)));
+        assertEquals(new Clock(22, 40), new Clock(-2, 40));
     }
 
     @Ignore("Remove to run test")
     @Test
     public void clocksWithNegativeHourThatWraps() {
-        assertTrue(new Clock(17, 3).equals(new Clock(-31, 3)));
+        assertEquals(new Clock(17, 3), new Clock(-31, 3));
     }
 
     @Ignore("Remove to run test")
     @Test
     public void clocksWithNegativeHourThatWrapsMultipleTimes() {
-        assertTrue(new Clock(13, 49).equals(new Clock(-83, 49)));
+        assertEquals(new Clock(13, 49), new Clock(-83, 49));
     }
 
     @Ignore("Remove to run test")
     @Test
     public void clocksWithMinuteOverflow() {
-        assertTrue(new Clock(0, 1).equals(new Clock(0, 1441)));
+        assertEquals(new Clock(0, 1), new Clock(0, 1441));
     }
 
     @Ignore("Remove to run test")
     @Test
     public void clocksWithMinuteOverflowBySeveralDays() {
-        assertTrue(new Clock(2, 2).equals(new Clock(2, 4322)));
+        assertEquals(new Clock(2, 2), new Clock(2, 4322));
     }
 
     @Ignore("Remove to run test")
     @Test
     public void clocksWithNegativeMinutes() {
-        assertTrue(new Clock(2, 40).equals(new Clock(3, -20)));
+        assertEquals(new Clock(2, 40), new Clock(3, -20));
     }
 
     @Ignore("Remove to run test")
     @Test
     public void clocksWithNegativeMinutesThatWraps() {
-        assertTrue(new Clock(4, 10).equals(new Clock(5, -1490)));
+        assertEquals(new Clock(4, 10), new Clock(5, -1490));
     }
 
     @Ignore("Remove to run test")
     @Test
     public void clocksWithNegativeMinutesThatWrapsMultipleTimes() {
-        assertTrue(new Clock(6, 15).equals(new Clock(6, -4305)));
+        assertEquals(new Clock(6, 15), new Clock(6, -4305));
     }
 
     @Ignore("Remove to run test")
     @Test
     public void clocksWithNegativeHoursAndMinutes() {
-        assertTrue(new Clock(7, 32).equals(new Clock(-12, -268)));
+        assertEquals(new Clock(7, 32), new Clock(-12, -268));
     }
 
     @Ignore("Remove to run test")
     @Test
     public void clocksWithNegativeHoursAndMinutesThatWrap() {
-        assertTrue(new Clock(18, 7).equals(new Clock(-54, -11513)));
+        assertEquals(new Clock(18, 7), new Clock(-54, -11513));
     }
+
 }