Skip to content
This repository was archived by the owner on Jun 9, 2021. It is now read-only.

Commit 338ae44

Browse files
committed
Fixing compile error in Travis
1 parent 305d071 commit 338ae44

File tree

3 files changed

+42
-42
lines changed

3 files changed

+42
-42
lines changed

src/test/java/se/bjurr/prnfb/listener/PrnfbPullRequestEventListenerTest.java

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434
import se.bjurr.prnfb.http.HttpResponse;
3535
import se.bjurr.prnfb.http.Invoker;
3636
import se.bjurr.prnfb.http.UrlInvoker;
37+
import se.bjurr.prnfb.service.MockedEscalatedSecurityContext;
3738
import se.bjurr.prnfb.service.PrnfbRenderer;
3839
import se.bjurr.prnfb.service.PrnfbRendererFactory;
3940
import se.bjurr.prnfb.service.SettingsService;
@@ -91,18 +92,9 @@ public String apply(UrlInvoker input) {
9192
public void before() throws ValidationException {
9293
initMocks(this);
9394
SecurityService securityService = mock(SecurityService.class);
94-
EscalatedSecurityContext escalatedSecurityContext = mock(EscalatedSecurityContext.class);
95+
EscalatedSecurityContext escalatedSecurityContext = new MockedEscalatedSecurityContext();
9596
when(securityService.withPermission(Mockito.any(), Mockito.any())) //
9697
.thenReturn(escalatedSecurityContext);
97-
when(escalatedSecurityContext.call(Mockito.any())) //
98-
.thenAnswer(
99-
new Answer<Boolean>() {
100-
@Override
101-
public Boolean answer(InvocationOnMock invocation) throws Throwable {
102-
return (Boolean) invocation.callRealMethod();
103-
}
104-
});
105-
10698
sut =
10799
new PrnfbPullRequestEventListener(
108100
prnfbRendererFactory,
Lines changed: 33 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,33 @@
1+
package se.bjurr.prnfb.service;
2+
3+
import java.util.Set;
4+
5+
import com.atlassian.bitbucket.permission.Permission;
6+
import com.atlassian.bitbucket.user.EscalatedSecurityContext;
7+
import com.atlassian.bitbucket.util.Operation;
8+
9+
public class MockedEscalatedSecurityContext implements EscalatedSecurityContext {
10+
11+
@Override
12+
public void applyToRequest() {}
13+
14+
@Override
15+
public <T, E extends Throwable> T call(Operation<T, E> arg0) throws E {
16+
return arg0.perform();
17+
}
18+
19+
@Override
20+
public EscalatedSecurityContext withPermission(Object arg0, Permission arg1) {
21+
return null;
22+
}
23+
24+
@Override
25+
public EscalatedSecurityContext withPermission(Permission arg0) {
26+
return null;
27+
}
28+
29+
@Override
30+
public EscalatedSecurityContext withPermissions(Set<Permission> arg0) {
31+
return null;
32+
}
33+
}

src/test/java/se/bjurr/prnfb/service/SettingsServiceTest.java

Lines changed: 7 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -13,27 +13,25 @@
1313
import static se.bjurr.prnfb.test.Podam.populatedInstanceOf;
1414

1515
import java.util.List;
16-
import java.util.Set;
1716

1817
import org.junit.Before;
1918
import org.junit.Test;
2019
import org.mockito.Mock;
2120

21+
import se.bjurr.prnfb.presentation.dto.ON_OR_OFF;
22+
import se.bjurr.prnfb.settings.PrnfbButton;
23+
import se.bjurr.prnfb.settings.PrnfbNotification;
24+
import se.bjurr.prnfb.settings.PrnfbSettings;
25+
import se.bjurr.prnfb.settings.ValidationException;
26+
2227
import com.atlassian.bitbucket.permission.Permission;
2328
import com.atlassian.bitbucket.user.EscalatedSecurityContext;
2429
import com.atlassian.bitbucket.user.SecurityService;
25-
import com.atlassian.bitbucket.util.Operation;
2630
import com.atlassian.sal.api.pluginsettings.PluginSettingsFactory;
2731
import com.atlassian.sal.api.transaction.TransactionCallback;
2832
import com.atlassian.sal.api.transaction.TransactionTemplate;
2933
import com.google.gson.Gson;
3034

31-
import se.bjurr.prnfb.presentation.dto.ON_OR_OFF;
32-
import se.bjurr.prnfb.settings.PrnfbButton;
33-
import se.bjurr.prnfb.settings.PrnfbNotification;
34-
import se.bjurr.prnfb.settings.PrnfbSettings;
35-
import se.bjurr.prnfb.settings.ValidationException;
36-
3735
public class SettingsServiceTest {
3836
private EscalatedSecurityContext escalatedSecurityContext;
3937
private PrnfbNotification notification1;
@@ -49,30 +47,7 @@ public void before() throws ValidationException {
4947
when(this.pluginSettingsFactory.createGlobalSettings()) //
5048
.thenReturn(this.pluginSettings);
5149
this.escalatedSecurityContext =
52-
new EscalatedSecurityContext() {
53-
@Override
54-
public void applyToRequest() {}
55-
56-
@Override
57-
public <T, E extends Throwable> T call(Operation<T, E> arg0) throws E {
58-
return arg0.perform();
59-
}
60-
61-
@Override
62-
public EscalatedSecurityContext withPermission(Object arg0, Permission arg1) {
63-
return null;
64-
}
65-
66-
@Override
67-
public EscalatedSecurityContext withPermission(Permission arg0) {
68-
return null;
69-
}
70-
71-
@Override
72-
public EscalatedSecurityContext withPermissions(Set<Permission> arg0) {
73-
return null;
74-
}
75-
};
50+
new MockedEscalatedSecurityContext();
7651
when(this.securityService.withPermission(Permission.ADMIN, "Getting config")) //
7752
.thenReturn(this.escalatedSecurityContext);
7853
this.transactionTemplate =

0 commit comments

Comments
 (0)