Fixed broken test.
This commit is contained in:
@@ -85,9 +85,9 @@ public class PreAuthTokenSourceTrustAuthenticationProviderTest {
|
||||
when(webAuthenticationDetailsMock.getRemoteAddress()).thenReturn(remoteAddress);
|
||||
|
||||
// test, should throw authentication exception
|
||||
final Authentication authenticate = underTestWithSourceIpCheck.authenticate(token);
|
||||
|
||||
try {
|
||||
assertThat(authenticate.isAuthenticated()).isTrue();
|
||||
underTestWithSourceIpCheck.authenticate(token);
|
||||
fail("as source is not trusted.");
|
||||
} catch (final InsufficientAuthenticationException e) {
|
||||
|
||||
|
||||
@@ -379,7 +379,6 @@ public class ControllerManagement {
|
||||
handleErrorOnAction(mergedAction, mergedTarget);
|
||||
break;
|
||||
case FINISHED:
|
||||
|
||||
handleFinishedAndStoreInTargetStatus(mergedTarget, mergedAction);
|
||||
break;
|
||||
case CANCELED:
|
||||
@@ -399,7 +398,6 @@ public class ControllerManagement {
|
||||
}
|
||||
|
||||
private void handleErrorOnAction(final Action mergedAction, final Target mergedTarget) {
|
||||
// set action inactive
|
||||
mergedAction.setActive(false);
|
||||
mergedAction.setStatus(Status.ERROR);
|
||||
mergedTarget.setAssignedDistributionSet(null);
|
||||
|
||||
Reference in New Issue
Block a user