Skip to content

Commit 6714355

Browse files
Experimental migration of JenkinsSessionRule and BuildWatcher
1 parent 53155f8 commit 6714355

File tree

2 files changed

+26
-25
lines changed

2 files changed

+26
-25
lines changed

pom.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,8 @@ THE SOFTWARE.
8989
<jenkins.version>${jenkins.baseline}.3</jenkins.version>
9090
<gitHubRepo>jenkinsci/throttle-concurrent-builds-plugin</gitHubRepo>
9191
<spotless.check.skip>false</spotless.check.skip>
92+
<!-- https://github.com/jenkinsci/jenkins-test-harness/pull/988 -->
93+
<jenkins-test-harness.version>2484.vca_2b_da_c8d4ca_</jenkins-test-harness.version>
9294
</properties>
9395

9496
<dependencyManagement>

src/test/java/hudson/plugins/throttleconcurrents/ThrottleJobPropertyPipelineRestartTest.java

Lines changed: 24 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -2,52 +2,51 @@
22

33
import static org.hamcrest.MatcherAssert.assertThat;
44
import static org.hamcrest.Matchers.hasItem;
5-
import static org.junit.Assert.assertEquals;
6-
import static org.junit.Assert.assertFalse;
7-
import static org.junit.Assert.assertNotNull;
8-
import static org.junit.Assert.assertTrue;
5+
import static org.junit.jupiter.api.Assertions.assertEquals;
6+
import static org.junit.jupiter.api.Assertions.assertFalse;
7+
import static org.junit.jupiter.api.Assertions.assertNotNull;
8+
import static org.junit.jupiter.api.Assertions.assertTrue;
99

1010
import hudson.model.Node;
1111
import hudson.model.Queue;
1212
import hudson.util.RunList;
13+
import java.io.File;
1314
import java.util.ArrayList;
1415
import java.util.Arrays;
1516
import java.util.Collections;
1617
import java.util.List;
1718
import java.util.Set;
18-
import java.util.stream.Collectors;
1919
import org.jenkinsci.plugins.workflow.job.WorkflowJob;
2020
import org.jenkinsci.plugins.workflow.job.WorkflowRun;
2121
import org.jenkinsci.plugins.workflow.test.steps.SemaphoreStep;
22-
import org.junit.ClassRule;
23-
import org.junit.Rule;
24-
import org.junit.Test;
25-
import org.junit.rules.TemporaryFolder;
26-
import org.jvnet.hudson.test.BuildWatcher;
27-
import org.jvnet.hudson.test.JenkinsSessionRule;
22+
import org.junit.jupiter.api.Test;
23+
import org.junit.jupiter.api.extension.RegisterExtension;
24+
import org.junit.jupiter.api.io.TempDir;
25+
import org.jvnet.hudson.test.junit.jupiter.BuildWatcherExtension;
26+
import org.jvnet.hudson.test.junit.jupiter.JenkinsSessionExtension;
2827

29-
public class ThrottleJobPropertyPipelineRestartTest {
28+
class ThrottleJobPropertyPipelineRestartTest {
3029

31-
@Rule
32-
public JenkinsSessionRule sessions = new JenkinsSessionRule();
30+
@RegisterExtension
31+
private final JenkinsSessionExtension sessions = new JenkinsSessionExtension();
3332

34-
@ClassRule
35-
public static BuildWatcher buildWatcher = new BuildWatcher();
33+
@RegisterExtension
34+
private static final BuildWatcherExtension buildWatcher = new BuildWatcherExtension();
3635

37-
@Rule
38-
public TemporaryFolder firstAgentTmp = new TemporaryFolder();
36+
@TempDir
37+
private File firstAgentTmp;
3938

40-
@Rule
41-
public TemporaryFolder secondAgentTmp = new TemporaryFolder();
39+
@TempDir
40+
private File secondAgentTmp;
4241

4342
@Test
44-
public void twoTotalWithRestart() throws Throwable {
43+
void twoTotalWithRestart() throws Throwable {
4544
String[] jobNames = new String[2];
4645
String[] agentNames = new String[2];
4746
sessions.then(j -> {
4847
List<Node> agents = new ArrayList<>();
49-
Node firstAgent = TestUtil.setupAgent(j, firstAgentTmp.getRoot(), agents, null, 4, "on-agent");
50-
Node secondAgent = TestUtil.setupAgent(j, secondAgentTmp.getRoot(), agents, null, 4, "on-agent");
48+
Node firstAgent = TestUtil.setupAgent(j, firstAgentTmp, agents, null, 4, "on-agent");
49+
Node secondAgent = TestUtil.setupAgent(j, secondAgentTmp, agents, null, 4, "on-agent");
5150
agentNames[0] = firstAgent.getNodeName();
5251
agentNames[1] = secondAgent.getNodeName();
5352
TestUtil.setupCategories(TestUtil.TWO_TOTAL);
@@ -106,7 +105,7 @@ public void twoTotalWithRestart() throws Throwable {
106105
j.jenkins.getQueue().maintain();
107106
assertFalse(j.jenkins.getQueue().isEmpty());
108107
List<Queue.Item> queuedItemList =
109-
Arrays.stream(j.jenkins.getQueue().getItems()).collect(Collectors.toList());
108+
Arrays.stream(j.jenkins.getQueue().getItems()).toList();
110109
assertEquals(1, queuedItemList.size());
111110
Queue.Item queuedItem = queuedItemList.get(0);
112111
Set<String> blockageReasons = TestUtil.getBlockageReasons(queuedItem.getCauseOfBlockage());
@@ -141,7 +140,7 @@ public void twoTotalWithRestart() throws Throwable {
141140

142141
assertFalse(j.jenkins.getQueue().isEmpty());
143142
List<Queue.Item> queuedItemList =
144-
Arrays.stream(j.jenkins.getQueue().getItems()).collect(Collectors.toList());
143+
Arrays.stream(j.jenkins.getQueue().getItems()).toList();
145144
assertEquals(1, queuedItemList.size());
146145
Queue.Item queuedItem = queuedItemList.get(0);
147146
Set<String> blockageReasons = TestUtil.getBlockageReasons(queuedItem.getCauseOfBlockage());

0 commit comments

Comments
 (0)