Skip to content

Commit c839141

Browse files
authored
Merge pull request #716 from rantoniuk/701-bump-to-next-lts
feat: Update Jenkins baseline version to 2.492.x
2 parents b62aae3 + 8402bb8 commit c839141

3 files changed

Lines changed: 11 additions & 12 deletions

File tree

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@
5353
<jira-rest-client.version>6.0.2</jira-rest-client.version>
5454
<fugue.version>4.7.2</fugue.version>
5555
<!-- https://www.jenkins.io/doc/developer/plugin-development/choosing-jenkins-baseline/ -->
56-
<jenkins.baseline>2.479</jenkins.baseline>
56+
<jenkins.baseline>2.492</jenkins.baseline>
5757
<jenkins.version>${jenkins.baseline}.3</jenkins.version>
5858
<spotless.check.skip>false</spotless.check.skip>
5959
</properties>

src/main/java/hudson/plugins/jira/pipeline/IssueFieldUpdateStep.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -138,15 +138,16 @@ public void submitFields(JiraSession session, String issueId, List<JiraIssueFiel
138138
} catch (RestClientException e) {
139139

140140
if (e.getStatusCode().or(0).equals(404)) {
141-
logger.println(issueId + " - Jira issue not found");
141+
logger.println("[Jira] " + issueId + " - Jira issue not found");
142142
}
143143

144144
if (e.getStatusCode().or(0).equals(403)) {
145-
logger.println(issueId + " - Jenkins Jira user does not have permissions to comment on this issue");
145+
logger.println("[Jira] " + issueId
146+
+ " - Jenkins Jira user does not have permissions to comment on this issue");
146147
}
147148

148149
if (e.getStatusCode().or(0).equals(401)) {
149-
logger.println(issueId + " - Jenkins Jira authentication problem");
150+
logger.println("[Jira] " + issueId + " - Jenkins Jira authentication problem");
150151
}
151152

152153
logger.println(Messages.FailedToUpdateIssue(issueId));

src/test/java/hudson/plugins/jira/UpdaterTest.java

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,10 @@
44
import static org.hamcrest.MatcherAssert.assertThat;
55
import static org.hamcrest.Matchers.containsString;
66
import static org.hamcrest.Matchers.equalTo;
7-
import static org.junit.jupiter.api.Assertions.*;
7+
import static org.junit.jupiter.api.Assertions.assertEquals;
8+
import static org.junit.jupiter.api.Assertions.assertNotNull;
9+
import static org.junit.jupiter.api.Assertions.assertThrows;
10+
import static org.junit.jupiter.api.Assertions.assertTrue;
811
import static org.mockito.ArgumentMatchers.anyString;
912
import static org.mockito.ArgumentMatchers.eq;
1013
import static org.mockito.Mockito.doAnswer;
@@ -38,7 +41,6 @@
3841
import java.util.List;
3942
import java.util.Locale;
4043
import java.util.Set;
41-
import jenkins.model.Jenkins;
4244
import org.hamcrest.Matchers;
4345
import org.jenkinsci.plugins.workflow.job.WorkflowJob;
4446
import org.jenkinsci.plugins.workflow.job.WorkflowRun;
@@ -295,12 +297,8 @@ void issueIsRemovedFromCarryOverListAfterSubmission() throws RestClientException
295297
*
296298
*/
297299
@Test
298-
@WithoutJenkins
299-
void getChangesUsingReflectionForWorkflowJob() throws IOException {
300-
Jenkins jenkins = mock(Jenkins.class);
301-
302-
when(jenkins.getRootDirFor(Mockito.any())).thenReturn(folder);
303-
WorkflowJob workflowJob = new WorkflowJob(jenkins, "job");
300+
void getChangesUsingReflectionForWorkflowJob(JenkinsRule r) throws IOException {
301+
WorkflowJob workflowJob = new WorkflowJob(r.getInstance(), "job");
304302
WorkflowRun workflowRun = new WorkflowRun(workflowJob);
305303

306304
ChangeLogSet.createEmpty(workflowRun);

0 commit comments

Comments
 (0)