From e2bbe7429bbe9cf0c42d4d7f492f592f837c646d Mon Sep 17 00:00:00 2001 From: akwizgran <akwizgran@users.sourceforge.net> Date: Wed, 16 Nov 2016 17:00:53 +0000 Subject: [PATCH] Moved some boilerplate into a field. --- .../clients/BdfMessageValidatorTest.java | 44 ++++++------------- 1 file changed, 14 insertions(+), 30 deletions(-) diff --git a/briar-tests/src/org/briarproject/clients/BdfMessageValidatorTest.java b/briar-tests/src/org/briarproject/clients/BdfMessageValidatorTest.java index 65d856dc00..1714b261b3 100644 --- a/briar-tests/src/org/briarproject/clients/BdfMessageValidatorTest.java +++ b/briar-tests/src/org/briarproject/clients/BdfMessageValidatorTest.java @@ -22,6 +22,17 @@ import static org.junit.Assert.fail; public class BdfMessageValidatorTest extends ValidatorTestCase { + private final BdfMessageValidator subclassNotCalled = + new BdfMessageValidator(clientHelper, metadataEncoder, clock) { + @Override + protected BdfMessageContext validateMessage(Message m, Group g, + BdfList body) + throws InvalidMessageException, FormatException { + fail(); + return null; + } + }; + private final BdfList body = BdfList.of(123, 456); private final BdfDictionary dictionary = new BdfDictionary(); private final Metadata meta = new Metadata(); @@ -37,16 +48,7 @@ public class BdfMessageValidatorTest extends ValidatorTestCase { will(returnValue(timestamp - MAX_CLOCK_DIFFERENCE - 1)); }}); - BdfMessageValidator v = new BdfMessageValidator(clientHelper, - metadataEncoder, clock) { - @Override - protected BdfMessageContext validateMessage(Message m, Group g, - BdfList b) throws InvalidMessageException, FormatException { - fail(); - return null; - } - }; - v.validateMessage(message, group); + subclassNotCalled.validateMessage(message, group); } @Test @@ -92,16 +94,7 @@ public class BdfMessageValidatorTest extends ValidatorTestCase { will(returnValue(invalidRaw)); }}); - BdfMessageValidator v = new BdfMessageValidator(clientHelper, - metadataEncoder, clock) { - @Override - protected BdfMessageContext validateMessage(Message m, Group g, - BdfList b) throws InvalidMessageException, FormatException { - fail(); - return null; - } - }; - v.validateMessage(invalidMessage, group); + subclassNotCalled.validateMessage(invalidMessage, group); } @Test @@ -146,16 +139,7 @@ public class BdfMessageValidatorTest extends ValidatorTestCase { will(throwException(new FormatException())); }}); - BdfMessageValidator v = new BdfMessageValidator(clientHelper, - metadataEncoder, clock) { - @Override - protected BdfMessageContext validateMessage(Message m, Group g, - BdfList b) throws InvalidMessageException, FormatException { - fail(); - return null; - } - }; - v.validateMessage(message, group); + subclassNotCalled.validateMessage(message, group); } @Test(expected = InvalidMessageException.class) -- GitLab