worked on code review coments

jbosstm · Apr 17, 2020 · 8d545e5 · 8d545e5
1 parent bf17d29
commit 8d545e5
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/ArjunaJTA/jta/tests/classes/com/hp/mwtests/ts/jta/recovery/RecoveryMonitorTest.java b/ArjunaJTA/jta/tests/classes/com/hp/mwtests/ts/jta/recovery/RecoveryMonitorTest.java
@@ -35,6 +35,7 @@
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.assertFalse;
 
 /**
  * Test the RecoveryMonitor is verbose mode
@@ -80,7 +81,7 @@ public void testRecoveryMonitorWithFailure() throws Exception {
             // check the output of the scan
             assertEquals("ERROR", RecoveryMonitor.getResponse());
             assertEquals("ERROR", RecoveryMonitor.getSystemOutput());
-            assertTrue(!xaRecoveryModule.isPeriodicWorkSuccessful());
+            assertFalse(xaRecoveryModule.isPeriodicWorkSuccessful());
         } finally {
             manager.terminate();
         }

diff --git a/ArjunaJTA/jta/tests/classes/com/hp/mwtests/ts/jta/recovery/RecoveryMonitorTest2.java b/ArjunaJTA/jta/tests/classes/com/hp/mwtests/ts/jta/recovery/RecoveryMonitorTest2.java
@@ -123,7 +123,7 @@ public void start(Xid xid, int i) throws XAException {
             // check the output of the scan
             assertEquals("DONE", RecoveryMonitor.getResponse());
             assertEquals("DONE", RecoveryMonitor.getSystemOutput());
-            assertFalse(!xaRecoveryModule.isPeriodicWorkSuccessful());
+            assertTrue(xaRecoveryModule.isPeriodicWorkSuccessful());
         } finally {
             manager.terminate();
         }

diff --git a/ArjunaJTA/jta/tests/classes/com/hp/mwtests/ts/jta/recovery/RecoveryScanTest.java b/ArjunaJTA/jta/tests/classes/com/hp/mwtests/ts/jta/recovery/RecoveryScanTest.java
@@ -93,7 +93,7 @@ public void testRecoveryMonitorWithFailure() throws Exception {
 
         manager.scan();
 
-        assertTrue(!xaRecoveryModule.isPeriodicWorkSuccessful());
+        assertFalse(xaRecoveryModule.isPeriodicWorkSuccessful());
     }
 
     @Test
@@ -158,7 +158,7 @@ public void start(Xid xid, int i) throws XAException {
 
         manager.scan();
 
-        assertFalse(!xaRecoveryModule.isPeriodicWorkSuccessful());
+        assertTrue(xaRecoveryModule.isPeriodicWorkSuccessful());
     }
 
     @Test