Skip to content

Commit 37b9011

Browse files
committed
final changes
1 parent dcc6cae commit 37b9011

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

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

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -255,8 +255,7 @@ static void verifySucceededCleanly(Jenkins j, WorkflowRun run) throws Exception
255255
* Currently dirty shutdowns have the potential to lose the head node ID if they happen at the moment of writing
256256
* it to build.xml
257257
*/
258-
static void verifyDirtyResumed(JenkinsRule rule, WorkflowRun run, String logStart) throws Exception {
259-
assert run.isBuilding();
258+
private static void verifyDirtyResumed(JenkinsRule rule, WorkflowRun run, String logStart) throws Exception {
260259
assertHasTimingAction(run.getExecution());
261260
rule.waitForCompletion(run);
262261
Assert.assertEquals(Result.SUCCESS, run.getResult());
@@ -266,8 +265,8 @@ static void verifyDirtyResumed(JenkinsRule rule, WorkflowRun run, String logStar
266265
}
267266

268267
/** If it's a {@link SemaphoreStep} we test less rigorously because that blocks async GraphListeners. */
269-
static void verifySafelyResumed(JenkinsRule rule, WorkflowRun run, boolean isSemaphore, String logStart) throws Exception {
270-
assert run.isBuilding();
268+
private static void verifySafelyResumed(JenkinsRule rule, WorkflowRun run, boolean isSemaphore, String logStart) throws Exception {
269+
Assert.assertTrue(run.isBuilding());
271270
FlowExecution exec = run.getExecution();
272271

273272
// Assert that we have the appropriate flow graph entries

0 commit comments

Comments
 (0)