From 61c05c1dd42891a2ff5999e404a1193e875481ad Mon Sep 17 00:00:00 2001 From: Torsten Grote <t@grobox.de> Date: Wed, 31 Aug 2016 11:32:53 -0300 Subject: [PATCH] Fix tests broken by latest merges! --- .../org/briarproject/blogs/BlogPostValidatorTest.java | 10 ++++++---- .../introduction/IntroductionValidatorTest.java | 3 ++- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/briar-tests/src/org/briarproject/blogs/BlogPostValidatorTest.java b/briar-tests/src/org/briarproject/blogs/BlogPostValidatorTest.java index fae1f04d7b..bdbcc2398c 100644 --- a/briar-tests/src/org/briarproject/blogs/BlogPostValidatorTest.java +++ b/briar-tests/src/org/briarproject/blogs/BlogPostValidatorTest.java @@ -37,6 +37,7 @@ import static org.briarproject.api.blogs.BlogConstants.KEY_AUTHOR; import static org.briarproject.api.blogs.BlogConstants.KEY_AUTHOR_ID; import static org.briarproject.api.blogs.BlogConstants.KEY_AUTHOR_NAME; import static org.briarproject.api.blogs.BlogConstants.KEY_COMMENT; +import static org.briarproject.api.blogs.BlogConstants.KEY_ORIGINAL_PARENT_MSG_ID; import static org.briarproject.api.blogs.BlogConstants.KEY_PARENT_MSG_ID; import static org.briarproject.api.blogs.BlogConstants.KEY_ORIGINAL_MSG_ID; import static org.briarproject.api.blogs.BlogConstants.KEY_PUBLIC_KEY; @@ -151,23 +152,24 @@ public class BlogPostValidatorTest extends BriarTestCase { throws IOException, GeneralSecurityException { // comment, parent_original_id, parent_id, signature String comment = "This is a blog comment"; - MessageId originalId = new MessageId(TestUtils.getRandomId()); + MessageId pOriginalId = new MessageId(TestUtils.getRandomId()); MessageId currentId = new MessageId(TestUtils.getRandomId()); final byte[] sigBytes = TestUtils.getRandomBytes(42); BdfList m = - BdfList.of(COMMENT.getInt(), comment, originalId, currentId, + BdfList.of(COMMENT.getInt(), comment, pOriginalId, currentId, sigBytes); BdfList signed = BdfList.of(blog.getId(), message.getTimestamp(), comment, - originalId, currentId); + pOriginalId, currentId); expectCrypto(signed, sigBytes, true); final BdfDictionary result = validator.validateMessage(message, group, m).getDictionary(); assertEquals(comment, result.getString(KEY_COMMENT)); assertEquals(authorDict, result.getDictionary(KEY_AUTHOR)); - assertEquals(originalId.getBytes(), result.getRaw(KEY_ORIGINAL_MSG_ID)); + assertEquals(pOriginalId.getBytes(), + result.getRaw(KEY_ORIGINAL_PARENT_MSG_ID)); assertEquals(currentId.getBytes(), result.getRaw(KEY_PARENT_MSG_ID)); assertFalse(result.getBoolean(KEY_READ)); context.assertIsSatisfied(); diff --git a/briar-tests/src/org/briarproject/introduction/IntroductionValidatorTest.java b/briar-tests/src/org/briarproject/introduction/IntroductionValidatorTest.java index fa0fee5e47..af9b2723cc 100644 --- a/briar-tests/src/org/briarproject/introduction/IntroductionValidatorTest.java +++ b/briar-tests/src/org/briarproject/introduction/IntroductionValidatorTest.java @@ -29,6 +29,7 @@ import static org.briarproject.api.introduction.IntroductionConstants.ACCEPT; import static org.briarproject.api.introduction.IntroductionConstants.E_PUBLIC_KEY; import static org.briarproject.api.introduction.IntroductionConstants.GROUP_ID; import static org.briarproject.api.introduction.IntroductionConstants.MAC; +import static org.briarproject.api.introduction.IntroductionConstants.MAC_LENGTH; import static org.briarproject.api.introduction.IntroductionConstants.MSG; import static org.briarproject.api.introduction.IntroductionConstants.NAME; import static org.briarproject.api.introduction.IntroductionConstants.PUBLIC_KEY; @@ -286,7 +287,7 @@ public class IntroductionValidatorTest extends BriarTestCase { @Test public void testValidateProperIntroductionAck() throws IOException { byte[] sessionId = TestUtils.getRandomId(); - byte[] mac = TestUtils.getRandomBytes(42); + byte[] mac = TestUtils.getRandomBytes(MAC_LENGTH); byte[] sig = TestUtils.getRandomBytes(MAX_SIGNATURE_LENGTH); BdfList body = BdfList.of(TYPE_ACK, sessionId, mac, sig); -- GitLab