Declare as final private classes 71/107671/3
authorGilles Thouenon <gilles.thouenon@orange.com>
Fri, 1 Sep 2023 17:06:08 +0000 (19:06 +0200)
committerGilles Thouenon <gilles.thouenon@orange.com>
Fri, 8 Sep 2023 15:33:06 +0000 (17:33 +0200)
Latest checkstyle version (10.12.3) reports bug when private class
without constructor is not declared final. Fix it up.

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

networkmodel/src/test/java/org/opendaylight/transportpce/networkmodel/util/OpenRoadmTopologyTest.java
tapi/src/test/java/org/opendaylight/transportpce/tapi/topology/TapiTopologyImplExceptionTest.java

index 5ca66393f99bca1982ba0ade6f86aa202b431967..31d9180ddf4cb8f229fb8b80a3ff3389ff15acef 100644 (file)
@@ -453,7 +453,7 @@ public class OpenRoadmTopologyTest {
         }
     }
 
-    private class InterruptedLinkFuture implements  ListenableFuture<Optional<Link>> {
+    private final class InterruptedLinkFuture implements  ListenableFuture<Optional<Link>> {
 
         @Override
         public boolean cancel(boolean arg0) {
index b09340b2582751a1964073bd7782392e443a9d8c..9e46f5395b3a2da46a5dae1afeeaec23302c7b0c 100644 (file)
@@ -75,7 +75,7 @@ public class TapiTopologyImplExceptionTest {
         }
     }
 
-    private class ReadTransactionMock implements ReadTransaction {
+    private final class ReadTransactionMock implements ReadTransaction {
 
         @Override
         public @NonNull Object getIdentifier() {