diff --git a/notification-core/src/test/java/org/opengroup/osdu/notification/api/PubsubEndpointTest.java b/notification-core/src/test/java/org/opengroup/osdu/notification/api/PubsubEndpointTest.java
index 75c9461fd3eddba5e0776cd65b87ed924cf78c49..ebc388a71ce54795ee0a9212085d817f2265a5d4 100644
--- a/notification-core/src/test/java/org/opengroup/osdu/notification/api/PubsubEndpointTest.java
+++ b/notification-core/src/test/java/org/opengroup/osdu/notification/api/PubsubEndpointTest.java
@@ -40,12 +40,16 @@ public class PubsubEndpointTest {
private static final String NOTIFICATION_ID = "test-notification-id";
private static final String PUBSUB_MESSAGE = "test-pubsub-message-data";
private final HttpResponse response = new HttpResponse();
+
@Mock
private IPubsubRequestBodyExtractor pubsubRequestBodyExtractor;
+
@Mock
private NotificationHandler notificationHandler;
+
@Mock
private JaxRsDpsLog log;
+
@InjectMocks
private PubsubEndpoint sut;
diff --git a/notification-core/src/test/java/org/opengroup/osdu/notification/auth/AuthorizationFilterTest.java b/notification-core/src/test/java/org/opengroup/osdu/notification/auth/AuthorizationFilterTest.java
index 485acc74bb86fda6417c266bec1bd59cb7c6dac9..44cfac89b8fb4d17fa2a8ceb07740d18aa3e93d2 100644
--- a/notification-core/src/test/java/org/opengroup/osdu/notification/auth/AuthorizationFilterTest.java
+++ b/notification-core/src/test/java/org/opengroup/osdu/notification/auth/AuthorizationFilterTest.java
@@ -53,16 +53,22 @@ public class AuthorizationFilterTest {
@Mock
private DpsHeaders headers;
+
@Mock
private HttpServletRequest request;
+
@Mock
private RequestInfoExt requestInfo;
+
@Mock
private IAuthorizationService authorizationService;
+
@Mock
private IServiceAccountValidator validator;
+
@Mock
private IPubsubRequestBodyExtractor extractor;
+
@InjectMocks
private AuthorizationFilter sut;
@@ -87,7 +93,6 @@ public class AuthorizationFilterTest {
public void should_throwAppError_when_noAuthzProvided() {
when(headers.getAuthorization()).thenReturn("");
final String USER_EMAIL = "test@test.com";
-
this.sut.hasAnyPermission(ROLE1, ROLE2);
assertEquals(USER_EMAIL, this.headers.getUserEmail());
}
diff --git a/notification-core/src/test/java/org/opengroup/osdu/notification/auth/AuthorizationServiceEntitlementsTest.java b/notification-core/src/test/java/org/opengroup/osdu/notification/auth/AuthorizationServiceEntitlementsTest.java
index 82895fd06ecc9a7735fca872b9aa113ef424f690..722c856aa9267b95ddfcae9252ec50641ced47b0 100644
--- a/notification-core/src/test/java/org/opengroup/osdu/notification/auth/AuthorizationServiceEntitlementsTest.java
+++ b/notification-core/src/test/java/org/opengroup/osdu/notification/auth/AuthorizationServiceEntitlementsTest.java
@@ -48,10 +48,13 @@ public class AuthorizationServiceEntitlementsTest {
@Mock
private IEntitlementsFactory entitlementsFactory;
+
@Mock
private IEntitlementsService service;
+
@Mock
private JaxRsDpsLog log;
+
@InjectMocks
private AuthorizationServiceImpl sut;
@@ -63,7 +66,6 @@ public class AuthorizationServiceEntitlementsTest {
@Test
public void should_returnUser_when_userHasPermission() throws EntitlementsException {
sut = createSut("service.register.user");
-
AuthorizationResponse result = sut.authorizeAny(DpsHeaders.createFromMap(new HashMap<>()), "service.register.user");
assertEquals("akelham@bbc.com", result.getUser());
@@ -72,7 +74,6 @@ public class AuthorizationServiceEntitlementsTest {
@Test
public void should_returnUser_when_userHasAnyPermission() throws EntitlementsException {
sut = createSut("service.register.editor");
-
AuthorizationResponse result = sut.authorizeAny(DpsHeaders.createFromMap(new HashMap<>()), "service.register.user", "service.register.editor");
assertEquals("akelham@bbc.com", result.getUser());
@@ -81,7 +82,6 @@ public class AuthorizationServiceEntitlementsTest {
@Test
public void should_throwUnauthorized_when_userDoesNotHaveRequiredPermission() throws EntitlementsException {
sut = createSut("service.register.user");
-
try {
sut.authorizeAny(DpsHeaders.createFromMap(new HashMap<>()), "service.register.editor");
fail("expected exception");
@@ -129,12 +129,9 @@ public class AuthorizationServiceEntitlementsTest {
Groups output = new Groups();
output.setMemberEmail("akelham@bbc.com");
output.setGroups(groupInfos);
-
when(service.getGroups()).thenReturn(output);
-
return sut;
}
-
}
diff --git a/notification-core/src/test/java/org/opengroup/osdu/notification/auth/GsaAuthTest.java b/notification-core/src/test/java/org/opengroup/osdu/notification/auth/GsaAuthTest.java
index 803fe3f540be0684116277a01de3825e2d6abad0..628e9f8339cc61073da9e9330a2baab95ad1e3d9 100644
--- a/notification-core/src/test/java/org/opengroup/osdu/notification/auth/GsaAuthTest.java
+++ b/notification-core/src/test/java/org/opengroup/osdu/notification/auth/GsaAuthTest.java
@@ -36,10 +36,13 @@ import static org.mockito.Mockito.when;
@RunWith(PowerMockRunner.class)
public class GsaAuthTest {
+
private static final String GOOGLE_ID_TOKEN = "testHeader.testPayload.testSignature";
private static Subscription gsa_subscription;
+
@Mock
private IGoogleServiceAccount gsaTokenProvider;
+
@InjectMocks
private GsaAuth sut;
diff --git a/notification-core/src/test/java/org/opengroup/osdu/notification/auth/HmacAuthTest.java b/notification-core/src/test/java/org/opengroup/osdu/notification/auth/HmacAuthTest.java
index 06ae5e76588bfc79a285242399e0913b8d135b10..406795fa388833faa95f120b55d6f8c576ae2baf 100644
--- a/notification-core/src/test/java/org/opengroup/osdu/notification/auth/HmacAuthTest.java
+++ b/notification-core/src/test/java/org/opengroup/osdu/notification/auth/HmacAuthTest.java
@@ -34,10 +34,13 @@ import static org.mockito.Mockito.when;
@RunWith(PowerMockRunner.class)
public class HmacAuthTest {
+
private static final String SIGNED_SIGNATURE = "testEncodedInfo.testSignature";
private static Subscription hmac_subscription;
+
@Mock
private ISignatureService signatureService;
+
@InjectMocks
private HmacAuth sut;
diff --git a/notification-core/src/test/java/org/opengroup/osdu/notification/auth/factory/AuthFactoryTest.java b/notification-core/src/test/java/org/opengroup/osdu/notification/auth/factory/AuthFactoryTest.java
index a1acb7de90ab752bf55a9a0afe2d5b03ce142b8a..5375c87c3ece41829ddc5fc94a8145cb7ca81ee9 100644
--- a/notification-core/src/test/java/org/opengroup/osdu/notification/auth/factory/AuthFactoryTest.java
+++ b/notification-core/src/test/java/org/opengroup/osdu/notification/auth/factory/AuthFactoryTest.java
@@ -28,14 +28,15 @@ import org.opengroup.osdu.notification.auth.interfaces.SecretAuth;
import static org.junit.Assert.assertTrue;
-
@RunWith(MockitoJUnitRunner.class)
public class AuthFactoryTest {
@InjectMocks
AuthFactory authFactory;
+
@Mock
private HmacAuth hmacAuth;
+
@Mock
private GsaAuth gsaAuth;
diff --git a/notification-core/src/test/java/org/opengroup/osdu/notification/di/CredentialHeadersProviderTest.java b/notification-core/src/test/java/org/opengroup/osdu/notification/di/CredentialHeadersProviderTest.java
index 83a349952ccb1cc50364408f0caa313d799bca4f..e7966e43b387cecb888a027c41d756d19ce9f8bf 100644
--- a/notification-core/src/test/java/org/opengroup/osdu/notification/di/CredentialHeadersProviderTest.java
+++ b/notification-core/src/test/java/org/opengroup/osdu/notification/di/CredentialHeadersProviderTest.java
@@ -22,6 +22,7 @@ import static org.mockito.Mockito.when;
@RunWith(MockitoJUnitRunner.class)
public class CredentialHeadersProviderTest {
+
@Mock
private HttpServletRequest httpRequest;
diff --git a/notification-core/src/test/java/org/opengroup/osdu/notification/di/RequestInfoExtTest.java b/notification-core/src/test/java/org/opengroup/osdu/notification/di/RequestInfoExtTest.java
index 3ea97c72192daa6e8c5fe99153a188c18d4d1cab..16e3f8c80c54d5ea0f55fba1be02854898fecf92 100644
--- a/notification-core/src/test/java/org/opengroup/osdu/notification/di/RequestInfoExtTest.java
+++ b/notification-core/src/test/java/org/opengroup/osdu/notification/di/RequestInfoExtTest.java
@@ -38,15 +38,21 @@ import static org.powermock.api.mockito.PowerMockito.when;
@RunWith(MockitoJUnitRunner.class)
public class RequestInfoExtTest {
+
+ Map<String, String> httpHeaders;
+
@Mock
DpsHeaders headers;
+
@Mock
IPubsubRequestBodyExtractor requestBodyExtractor;
- Map<String, String> httpHeaders;
+
@Mock
private IAppProperties config;
+
@Mock
private HttpServletRequest http;
+
@Spy
@InjectMocks
private RequestInfoExt sut;
@@ -61,7 +67,6 @@ public class RequestInfoExtTest {
MockitoAnnotations.initMocks(RequestInfoExtTest.this);
}
-
@Test
public void assignPartitionIdIfNotInHeaderTest() {
when(http.getMethod()).thenReturn("POST");
diff --git a/notification-core/src/test/java/org/opengroup/osdu/notification/logging/ResponseLogFilterTest.java b/notification-core/src/test/java/org/opengroup/osdu/notification/logging/ResponseLogFilterTest.java
index c78c8743fd17fa3f859c650ef863edf720ec7002..945ef7fda025ee39b27e46ed2a241549a5f2115a 100644
--- a/notification-core/src/test/java/org/opengroup/osdu/notification/logging/ResponseLogFilterTest.java
+++ b/notification-core/src/test/java/org/opengroup/osdu/notification/logging/ResponseLogFilterTest.java
@@ -80,6 +80,5 @@ public class ResponseLogFilterTest {
when(servletRequest.getMethod()).thenReturn("OPTIONS");
responseLogFilter.doFilter(servletRequest, servletResponse, filterChain);
verify(logger, times(1)).request(eq("notification.request"), any(Request.class), anyMap());
-
}
}
diff --git a/notification-core/src/test/java/org/opengroup/osdu/notification/service/NotificationHandlerTest.java b/notification-core/src/test/java/org/opengroup/osdu/notification/service/NotificationHandlerTest.java
index 783597c911e3bdabf02eedc7db76f5cde8ae652b..3336ef1f2989c40ae27be95682b0ba33e91ae401 100644
--- a/notification-core/src/test/java/org/opengroup/osdu/notification/service/NotificationHandlerTest.java
+++ b/notification-core/src/test/java/org/opengroup/osdu/notification/service/NotificationHandlerTest.java
@@ -43,23 +43,30 @@ import static org.mockito.Mockito.when;
@RunWith(MockitoJUnitRunner.class)
public class NotificationHandlerTest {
+
private static final String NOTIFICATION_ID = "test-notification-id";
private static final String PUBSUB_MESSAGE = "test-pubsub-message-data";
private static Subscription gsa_subscription;
private static Subscription hmac_subscription;
+ private final HttpResponse response = new HttpResponse();
+
@Mock
private SubscriptionHandler subscriptionHandler;
+
@Mock
private AuthFactory authFactory;
+
@Mock
private SecretAuth secretAuth;
+
@Mock
private HttpClient httpClient;
+
@Mock
private JaxRsDpsLog log;
+
@InjectMocks
private NotificationHandler sut;
- private final HttpResponse response = new HttpResponse();
@BeforeClass
public static void setup() {
diff --git a/notification-core/src/test/java/org/opengroup/osdu/notification/service/SubscriptionHandlerTest.java b/notification-core/src/test/java/org/opengroup/osdu/notification/service/SubscriptionHandlerTest.java
index f917f1fe9282d4f386a93cd0e03561e64ea2a949..f90a0cf904709b69a504dc439c63c1a9d07ff502 100644
--- a/notification-core/src/test/java/org/opengroup/osdu/notification/service/SubscriptionHandlerTest.java
+++ b/notification-core/src/test/java/org/opengroup/osdu/notification/service/SubscriptionHandlerTest.java
@@ -42,13 +42,18 @@ import static org.mockito.Mockito.when;
@RunWith(MockitoJUnitRunner.class)
public class SubscriptionHandlerTest {
+
private static final String NOTIFICATION_ID = "test-notification-id";
+
@Mock
private ISubscriptionFactory subscriptionFactory;
+
@Mock
private SubscriptionCacheFactory subscriptionCacheFactory;
+
@Mock
private ObjectMapper objectMapper;
+
@InjectMocks
private SubscriptionHandler sut;
diff --git a/notification-core/src/test/java/org/opengroup/osdu/notification/utils/NotificationFilterTest.java b/notification-core/src/test/java/org/opengroup/osdu/notification/utils/NotificationFilterTest.java
index ed6d109d71ef49d91b1f59983251400323b0f967..ae67d0617d868da34cb3a0d6b059682d67b394bb 100644
--- a/notification-core/src/test/java/org/opengroup/osdu/notification/utils/NotificationFilterTest.java
+++ b/notification-core/src/test/java/org/opengroup/osdu/notification/utils/NotificationFilterTest.java
@@ -63,7 +63,6 @@ public class NotificationFilterTest {
@Test
public void doFilterTest() throws ServletException, IOException {
-
notificationFilter.doFilter(servletRequest, servletResponse, filterChain);
assertNotNull(dpsHeaders.getHeaders().get("correlation-id"));
assertEquals(dpsHeaders.getHeaders().get("content-type"), "application/json");