Skip to content

Commit c8aa77a

Browse files
committed
Incorporate feedback from @basil
1 parent ba13afb commit c8aa77a

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

src/test/java/org/jenkinsci/plugins/workflow/cps/CpsFlowDefinitionTest.java

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,9 @@
22

33
import org.junit.Test;
44

5+
import static org.junit.Assert.assertTrue;
6+
import static org.junit.Assert.assertEquals;
7+
58
/**
69
* @author Kohsuke Kawaguchi
710
*/
@@ -19,7 +22,7 @@ public void simplestPossibleTest() throws Exception {
1922
exec.start();
2023
exec.waitForSuspension();
2124

22-
assert exec.isComplete();
25+
assertTrue(exec.isComplete());
2326
}
2427

2528
@Test
@@ -34,10 +37,10 @@ public void exceptionInWorkflowShouldBreakFlowExecution() throws Exception {
3437

3538
// it should stop at watch and suspend.
3639
exec.waitForSuspension();
37-
assert exec.isComplete();
40+
assertTrue(exec.isComplete());
3841
Throwable t = exec.getCauseOfFailure();
39-
assert t.getClass().equals(Throwable.class);
40-
assert t.getMessage().equals("This is a fire drill, not a real fire");
42+
assertEquals(Throwable.class, t.getClass());
43+
assertEquals("This is a fire drill, not a real fire", t.getMessage());
4144
}
4245

4346
}

0 commit comments

Comments
 (0)