Fix small issues in nbinotifications module 71/104371/3
authorGilles Thouenon <gilles.thouenon@orange.com>
Sun, 12 Feb 2023 08:19:26 +0000 (09:19 +0100)
committerGilles Thouenon <gilles.thouenon@orange.com>
Tue, 14 Feb 2023 15:14:07 +0000 (15:14 +0000)
- remove unused property in TopicManager
- remove unused variable in PublisherTest

Signed-off-by: Gilles Thouenon <gilles.thouenon@orange.com>
Change-Id: I92a63c4df337891724bfa08bb7fcebf0eda7da4e

nbinotifications/src/main/java/org/opendaylight/transportpce/nbinotifications/utils/TopicManager.java
nbinotifications/src/test/java/org/opendaylight/transportpce/nbinotifications/producer/PublisherTest.java

index ea596de38d1e275af3eb28333a127f6adf373f3f..dad51171fcf151036b72de4d31bd357183b9e876 100644 (file)
@@ -34,7 +34,6 @@ public final class TopicManager {
     private Map<String, Publisher<NotificationProcessService>> processPublisherMap = new HashMap<>();
     private JsonStringConverter<NotificationProcessService> processConverter;
     private JsonStringConverter<NotificationAlarmService> alarmConverter;
-    private int calledSetConverter = 0;
 
     private TopicManager() {
     }
@@ -57,7 +56,6 @@ public final class TopicManager {
 
     public void setTapiConverter(JsonStringConverter<NotificationTapiService> tapiConverter) {
         this.tapiConverter = tapiConverter;
-        this.calledSetConverter++;
     }
 
     public void setPublisherServer(String publisherServer) {
index 0e07aff231382b9e25252ff41d30eae7b5ffa9b5..507fb9e043f4b3dd9a7b5f7dc08f06d3b24f79ff 100644 (file)
@@ -9,7 +9,6 @@ package org.opendaylight.transportpce.nbinotifications.producer;
 
 import static org.junit.Assert.assertEquals;
 
-import com.google.common.util.concurrent.ListenableFuture;
 import java.io.IOException;
 import java.nio.file.Files;
 import java.nio.file.Paths;
@@ -39,10 +38,8 @@ import org.opendaylight.yang.gen.v1.nbi.notifications.rev211013.NotificationProc
 import org.opendaylight.yang.gen.v1.nbi.notifications.rev211013.NotificationTapiService;
 import org.opendaylight.yang.gen.v1.urn.onf.otcc.yang.tapi.common.rev181210.Uuid;
 import org.opendaylight.yang.gen.v1.urn.onf.otcc.yang.tapi.notification.rev181210.CreateNotificationSubscriptionServiceInputBuilder;
-import org.opendaylight.yang.gen.v1.urn.onf.otcc.yang.tapi.notification.rev181210.CreateNotificationSubscriptionServiceOutput;
 import org.opendaylight.yang.gen.v1.urn.onf.otcc.yang.tapi.notification.rev181210.create.notification.subscription.service.input.SubscriptionFilter;
 import org.opendaylight.yang.gen.v1.urn.onf.otcc.yang.tapi.notification.rev181210.create.notification.subscription.service.input.SubscriptionFilterBuilder;
-import org.opendaylight.yangtools.yang.common.RpcResult;
 import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
 import org.opendaylight.yangtools.yang.data.codec.gson.JSONCodecFactorySupplier;
 
@@ -122,8 +119,7 @@ public class PublisherTest extends AbstractTest {
             .setRequestedObjectIdentifier(new HashSet<>(List.of(new Uuid("76d8f07b-ead5-4132-8eb8-cf3fdef7e079"))))
             .build();
         builder.setSubscriptionFilter(subscriptionFilter);
-        ListenableFuture<RpcResult<CreateNotificationSubscriptionServiceOutput>> result =
-            nbiNotificationsImpl.createNotificationSubscriptionService(builder.build());
+        nbiNotificationsImpl.createNotificationSubscriptionService(builder.build());
         String json = Files.readString(Paths.get("src/test/resources/tapi_event.json"));
         NotificationTapiService notificationTapiService = converterTapiService
             .createDataObjectFromJsonString(YangInstanceIdentifier.of(NotificationTapiService.QNAME),