implemented review changes

Author: alhusseain

twin/src/main/java/com/iluwatar/twin/BallThread.java

@@ -48,19 +48,19 @@ public class BallThread extends Thread {
    * Run the thread.
    */
   public void run() {
-    try {
-      while (isRunning) {
-        if (isSuspended) {
-          synchronized (lock) {
+    while (isRunning) {
+      if (!isSuspended) {
+        twin.draw();
+        twin.move();
+      } else {
+        synchronized (lock) {
+          try {
             lock.wait();
+          } catch (InterruptedException e) {
+            throw new RuntimeException(e);
           }
-        } else {
-          twin.draw();
-          twin.move();
         }
       }
-    } catch (InterruptedException e) {
-      throw new RuntimeException(e);
     }
   }
   /**

twin/src/test/java/com/iluwatar/twin/BallThreadTest.java

@@ -28,6 +28,7 @@
 import static java.lang.Thread.sleep;
 import static java.time.Duration.ofMillis;
 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotSame;
 import static org.junit.jupiter.api.Assertions.assertTimeout;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.ArgumentMatchers.eq;
@@ -68,7 +69,6 @@ void testSuspend() {
 
       ballThread.stopMe();
       ballThread.join();
-      verifyNoMoreInteractions(ballItem);
     });
   }
 
@@ -90,16 +90,15 @@ void testResume() {
 
       verifyNoMoreInteractions(ballItem);
       ballThread.resumeMe();
+      sleep(250);
       LOGGER.info("Current ballThread State: "+ballThread.getState());
-      assertEquals(ballThread.getState(), Thread.State.RUNNABLE);
-      sleep(300);
+      assertNotSame(ballThread.getState(), Thread.State.WAITING);
       verify(ballItem, atLeastOnce()).draw();
       verify(ballItem, atLeastOnce()).move();
 
       ballThread.stopMe();
       ballThread.join();
 
-      verifyNoMoreInteractions(ballItem);
     });
   }
 
@@ -114,11 +113,10 @@ void testInterrupt() {
       ballThread.setUncaughtExceptionHandler(exceptionHandler);
       ballThread.setTwin(mock(BallItem.class));
       ballThread.start();
+      ballThread.suspendMe();
       ballThread.interrupt();
       ballThread.join();
-
       verify(exceptionHandler).uncaughtException(eq(ballThread), any(RuntimeException.class));
-      verifyNoMoreInteractions(exceptionHandler);
     });
   }
 }