From 847b6e41795e4c79642016508238dcb57b854d09 Mon Sep 17 00:00:00 2001
From: akwizgran <akwizgran@users.sourceforge.net>
Date: Fri, 4 Nov 2016 09:31:49 +0000
Subject: [PATCH] Added comments to integration test.

---
 .../test/java/org/briarproject/PrivateGroupManagerTest.java | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/briar-android-tests/src/test/java/org/briarproject/PrivateGroupManagerTest.java b/briar-android-tests/src/test/java/org/briarproject/PrivateGroupManagerTest.java
index d7dc816ed8..2b07de3452 100644
--- a/briar-android-tests/src/test/java/org/briarproject/PrivateGroupManagerTest.java
+++ b/briar-android-tests/src/test/java/org/briarproject/PrivateGroupManagerTest.java
@@ -392,6 +392,7 @@ public class PrivateGroupManagerTest extends BriarIntegrationTest {
 				privateGroup0.getId());
 		byte[] creatorSignature =
 				clientHelper.sign(toSign, author0.getPrivateKey());
+		// join message should not include invite time and creator's signature
 		GroupMessage joinMsg0 = groupMessageFactory
 				.createJoinMessage(privateGroup0.getId(), joinTime, author0,
 						inviteTime, creatorSignature);
@@ -407,7 +408,7 @@ public class PrivateGroupManagerTest extends BriarIntegrationTest {
 		t0.getDatabaseComponent().commitTransaction(txn0);
 		t0.getDatabaseComponent().endTransaction(txn0);
 
-		// author1 joins privateGroup0 with wrong own signature
+		// author1 joins privateGroup0 with wrong signature in join message
 		joinTime = clock.currentTimeMillis();
 		inviteTime = joinTime - 1;
 		invitationGroup = contactGroupFactory
@@ -415,6 +416,7 @@ public class PrivateGroupManagerTest extends BriarIntegrationTest {
 						author0.getId(), author1.getId());
 		toSign = BdfList.of(0, inviteTime, invitationGroup.getId(),
 				privateGroup0.getId());
+		// signature uses joiner's key, not creator's key
 		creatorSignature = clientHelper.sign(toSign, author1.getPrivateKey());
 		GroupMessage joinMsg1 = groupMessageFactory
 				.createJoinMessage(privateGroup0.getId(), joinTime, author1,
@@ -531,7 +533,7 @@ public class PrivateGroupManagerTest extends BriarIntegrationTest {
 		t0.getDatabaseComponent()
 				.setVisibleToContact(txn0, contactId1, privateGroup0.getId(),
 						true);
-		t0.getDatabaseComponent().commitTransaction(txn0);;
+		t0.getDatabaseComponent().commitTransaction(txn0);
 		t0.getDatabaseComponent().endTransaction(txn0);
 
 		// author1 joins privateGroup0
-- 
GitLab