Skip to content
Snippets Groups Projects
Commit 8dfbba28 authored by OZGCloud's avatar OZGCloud
Browse files

make SonarQube more happy; adjust Test class name to file name

parent 350c794e
Branches
Tags
No related merge requests found
...@@ -85,7 +85,7 @@ public abstract class UserResourceMapper { ...@@ -85,7 +85,7 @@ public abstract class UserResourceMapper {
.map(attributes -> attributes.get(properties.ldapIdKey())) .map(attributes -> attributes.get(properties.ldapIdKey()))
.filter(Objects::nonNull) .filter(Objects::nonNull)
.map(id -> id.get(0)) .map(id -> id.get(0))
.orElseGet(() -> userRepresentation.getId()); .orElseGet(userRepresentation::getId);
} }
......
...@@ -39,7 +39,7 @@ public class JwtUtil { ...@@ -39,7 +39,7 @@ public class JwtUtil {
public List<String> getOrganisationsEinheitIds() { public List<String> getOrganisationsEinheitIds() {
return Optional.ofNullable(jwt.getClaim(ORGANSIATIONSEINHEIT_IDS_CLAIM)) return Optional.ofNullable(jwt.getClaim(ORGANSIATIONSEINHEIT_IDS_CLAIM))
.map(List.class::cast) .map(List.class::cast)
.orElseGet(() -> Collections.emptyList()); .orElseGet(Collections::emptyList);
} }
public Optional<String> getUserId() { public Optional<String> getUserId() {
...@@ -61,8 +61,8 @@ public class JwtUtil { ...@@ -61,8 +61,8 @@ public class JwtUtil {
.map(Map.class::cast) .map(Map.class::cast)
.map(clientMap -> clientMap.get(MAP_ROLES_KEY)) .map(clientMap -> clientMap.get(MAP_ROLES_KEY))
.map(JsonArray.class::cast) .map(JsonArray.class::cast)
.map(rolesArray -> getAsStringArray(rolesArray)) .map(this::getAsStringArray)
.orElseGet(() -> Collections.<String>emptyList()); .orElseGet(Collections::<String>emptyList);
} }
private List<String> getAsStringArray(JsonArray rolesArray) { private List<String> getAsStringArray(JsonArray rolesArray) {
......
package de.itvsh.kop.user.settings; package de.itvsh.kop.user.settings;
import java.util.Optional;
import javax.enterprise.context.ApplicationScoped; import javax.enterprise.context.ApplicationScoped;
import javax.inject.Inject; import javax.inject.Inject;
import javax.ws.rs.core.Link; import javax.ws.rs.core.Link;
...@@ -39,11 +37,7 @@ class UserSettingsResourceAssembler { ...@@ -39,11 +37,7 @@ class UserSettingsResourceAssembler {
} }
boolean hasValidEmail() { boolean hasValidEmail() {
return Optional.ofNullable(getCurrentCallContextUser()).map(user -> user.getEmail()).filter(StringUtils::isNotEmpty).isPresent(); return currentUserService.getCurrentCallContextUser().map(CallContextUser::getEmail).filter(StringUtils::isNotEmpty).isPresent();
}
private CallContextUser getCurrentCallContextUser() {
return currentUserService.getCurrentCallContextUser().orElseGet(null);
} }
boolean hasVerwaltungUserRole() { boolean hasVerwaltungUserRole() {
......
...@@ -60,8 +60,7 @@ class JwtUtilTest { ...@@ -60,8 +60,7 @@ class JwtUtilTest {
var userName = jwtUtil.getUserName(); var userName = jwtUtil.getUserName();
assertThat(userName).isPresent(); assertThat(userName).contains(userNameInClaim);
assertThat(userName.get()).isEqualTo(userNameInClaim);
} }
@Test @Test
...@@ -129,8 +128,7 @@ class JwtUtilTest { ...@@ -129,8 +128,7 @@ class JwtUtilTest {
var userId = jwtUtil.getUserId(); var userId = jwtUtil.getUserId();
assertThat(userId).isPresent(); assertThat(userId).contains(userIdInClaim);
assertThat(userId.get()).isEqualTo(userIdInClaim);
} }
@Test @Test
...@@ -161,7 +159,7 @@ class JwtUtilTest { ...@@ -161,7 +159,7 @@ class JwtUtilTest {
var email = jwtUtil.getEmail(); var email = jwtUtil.getEmail();
assertThat(email).isEqualTo(email); assertThat(email).isEqualTo(emailInClaim);
} }
@Test @Test
...@@ -170,7 +168,7 @@ class JwtUtilTest { ...@@ -170,7 +168,7 @@ class JwtUtilTest {
var email = jwtUtil.getEmail(); var email = jwtUtil.getEmail();
assertThat(email).isEqualTo(null); assertThat(email).isNull();
} }
} }
......
...@@ -17,7 +17,7 @@ import de.itvsh.kop.user.common.callcontext.GrpcCallContextInterceptor.LogContex ...@@ -17,7 +17,7 @@ import de.itvsh.kop.user.common.callcontext.GrpcCallContextInterceptor.LogContex
import io.grpc.Metadata; import io.grpc.Metadata;
import io.grpc.ServerCall; import io.grpc.ServerCall;
class CallContextHandleInterceptorTest { class GrpcCallContextInterceptorTest {
@InjectMocks @InjectMocks
private GrpcCallContextInterceptor interceptor; private GrpcCallContextInterceptor interceptor;
...@@ -159,8 +159,7 @@ class CallContextHandleInterceptorTest { ...@@ -159,8 +159,7 @@ class CallContextHandleInterceptorTest {
@Test @Test
void shouldClearSecurityContextOnException() { void shouldClearSecurityContextOnException() {
var testException = new RuntimeException() { var testException = new RuntimeException();
};
assertThatThrownBy(() -> listener.doSurroundOn(() -> { assertThatThrownBy(() -> listener.doSurroundOn(() -> {
throw testException; throw testException;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment