Skip to content

Commit 0926136

Browse files
jglickbitwiseman
andauthored
Let BitbucketEndpointConfigurationTest tolerate Spring Security version of AccessDeniedException (#362)
* Let BitbucketEndpointConfigurationTest tolerate Spring Security version of AccessDeniedException * Use gitHubRepo Co-authored-by: Liam Newman <[email protected]>
1 parent fc5795e commit 0926136

File tree

3 files changed

+8
-6
lines changed

3 files changed

+8
-6
lines changed

.mvn/extensions.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,6 @@
22
<extension>
33
<groupId>io.jenkins.tools.incrementals</groupId>
44
<artifactId>git-changelist-maven-extension</artifactId>
5-
<version>1.0-beta-4</version>
5+
<version>1.2</version>
66
</extension>
77
</extensions>

pom.xml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@
5151
<properties>
5252
<revision>2.9.3</revision>
5353
<changelist>-SNAPSHOT</changelist>
54+
<gitHubRepo>jenkinsci/bitbucket-branch-source-plugin</gitHubRepo>
5455
<jenkins.version>2.176.4</jenkins.version>
5556
<java.level>8</java.level>
5657
<hpi.compatibleSinceVersion>2.0</hpi.compatibleSinceVersion>
@@ -76,9 +77,9 @@
7677
</developers>
7778

7879
<scm>
79-
<connection>scm:git:git://github.com/jenkinsci/bitbucket-branch-source-plugin.git</connection>
80-
<developerConnection>scm:git:[email protected]:jenkinsci/bitbucket-branch-source-plugin.git</developerConnection>
81-
<url>https://github.com/jenkinsci/bitbucket-branch-source-plugin</url>
80+
<connection>scm:git:git://github.com/${gitHubRepo}.git</connection>
81+
<developerConnection>scm:git:[email protected]:${gitHubRepo}.git</developerConnection>
82+
<url>https://github.com/${gitHubRepo}</url>
8283
<tag>${scmTag}</tag>
8384
</scm>
8485

src/test/java/com/cloudbees/jenkins/plugins/bitbucket/endpoints/BitbucketEndpointConfigurationTest.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,6 @@
4545
import java.util.Collections;
4646
import java.util.List;
4747
import jenkins.model.Jenkins;
48-
import org.acegisecurity.AccessDeniedException;
4948
import org.junit.Before;
5049
import org.junit.ClassRule;
5150
import org.junit.Test;
@@ -110,7 +109,7 @@ public void given__newInstance__when__configuredWithMultipleCloud__then__onlyFir
110109
assertThat(instance.getEndpoints().get(0).getCredentialsId(), is("first"));
111110
}
112111

113-
@Test(expected = AccessDeniedException.class)
112+
@Test
114113
public void given__newInstance__when__configuredAsAnon__then__permissionError() {
115114
BitbucketEndpointConfiguration instance = new BitbucketEndpointConfiguration();
116115
j.jenkins.setAuthorizationStrategy(new FullControlOnceLoggedInAuthorizationStrategy());
@@ -121,6 +120,8 @@ public void given__newInstance__when__configuredAsAnon__then__permissionError()
121120
new BitbucketCloudEndpoint(true, "third")));
122121
assertThat(instance.getEndpoints(), contains(instanceOf(BitbucketCloudEndpoint.class)));
123122
assertThat(instance.getEndpoints().get(0).getCredentialsId(), is("first"));
123+
} catch (RuntimeException x) {
124+
assertThat(x.getMessage(), is(hudson.security.Messages.AccessDeniedException2_MissingPermission("anonymous", "Overall/Administer")));
124125
} finally {
125126
j.jenkins.setAuthorizationStrategy(AuthorizationStrategy.UNSECURED);
126127
}

0 commit comments

Comments
 (0)