X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fcds-access-api%2Fsrc%2Ftest%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Faccess%2Fcommands%2FAbstractRequestFailureTest.java;h=78456b246afaf9014776b047ac33dfec48952bed;hb=HEAD;hp=69c9ea263f54361c1f989dbcfc7b7a19631670f1;hpb=54e20827dff665094aa1d187124e79e855439a96;p=controller.git diff --git a/opendaylight/md-sal/cds-access-api/src/test/java/org/opendaylight/controller/cluster/access/commands/AbstractRequestFailureTest.java b/opendaylight/md-sal/cds-access-api/src/test/java/org/opendaylight/controller/cluster/access/commands/AbstractRequestFailureTest.java index 69c9ea263f..78456b246a 100644 --- a/opendaylight/md-sal/cds-access-api/src/test/java/org/opendaylight/controller/cluster/access/commands/AbstractRequestFailureTest.java +++ b/opendaylight/md-sal/cds-access-api/src/test/java/org/opendaylight/controller/cluster/access/commands/AbstractRequestFailureTest.java @@ -7,8 +7,11 @@ */ package org.opendaylight.controller.cluster.access.commands; -import org.apache.commons.lang.SerializationUtils; -import org.junit.Assert; +import static java.util.Objects.requireNonNull; +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertTrue; + +import org.apache.commons.lang3.SerializationUtils; import org.junit.Test; import org.opendaylight.controller.cluster.access.concepts.ClientIdentifier; import org.opendaylight.controller.cluster.access.concepts.FrontendIdentifier; @@ -20,7 +23,7 @@ import org.opendaylight.controller.cluster.access.concepts.RequestFailure; import org.opendaylight.controller.cluster.access.concepts.RuntimeRequestException; import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier; -public abstract class AbstractRequestFailureTest { +public abstract class AbstractRequestFailureTest> { private static final FrontendIdentifier FRONTEND_IDENTIFIER = FrontendIdentifier.create( MemberName.forName("member"), FrontendType.forName("frontend")); @@ -29,26 +32,36 @@ public abstract class AbstractRequestFailureTest { protected static final TransactionIdentifier TRANSACTION_IDENTIFIER = new TransactionIdentifier( HISTORY_IDENTIFIER, 0); protected static final RequestException CAUSE = new RuntimeRequestException("fail", new Throwable()); + private static final int CAUSE_SIZE = SerializationUtils.serialize(CAUSE).length; + + private final T object; + private final int expectedSize; - abstract T object(); + protected AbstractRequestFailureTest(final T object, final int baseSize) { + this.object = requireNonNull(object); + this.expectedSize = baseSize + CAUSE_SIZE; + } @Test - public void getCauseTest() throws Exception { - Assert.assertEquals(CAUSE, object().getCause()); + public void getCauseTest() { + assertEquals(CAUSE, object.getCause()); } @Test - public void isHardFailureTest() throws Exception { - Assert.assertTrue(object().isHardFailure()); + public void isHardFailureTest() { + assertTrue(object.isHardFailure()); } - @SuppressWarnings("unchecked") @Test public void serializationTest() { - final Object deserialize = SerializationUtils.clone(object()); + final var bytes = SerializationUtils.serialize(object); + assertEquals(expectedSize, bytes.length); + + @SuppressWarnings("unchecked") + final var deserialize = (T) SerializationUtils.deserialize(bytes); - Assert.assertEquals(object().getTarget(), ((T) deserialize).getTarget()); - Assert.assertEquals(object().getVersion(), ((T) deserialize).getVersion()); - Assert.assertEquals(object().getSequence(), ((T) deserialize).getSequence()); + assertEquals(object.getTarget(), deserialize.getTarget()); + assertEquals(object.getVersion(), deserialize.getVersion()); + assertEquals(object.getSequence(), deserialize.getSequence()); } }