Address review comments.

This commit is contained in:
akwizgran
2019-05-22 17:46:17 +01:00
parent 01ef367864
commit de97cce119
2 changed files with 6 additions and 13 deletions

View File

@@ -10,7 +10,6 @@ import org.briarproject.bramble.api.contact.ContactManager;
import org.briarproject.bramble.api.contact.PendingContact;
import org.briarproject.bramble.api.contact.PendingContactId;
import org.briarproject.bramble.api.contact.PendingContactState;
import org.briarproject.bramble.api.contact.event.PendingContactAddedEvent;
import org.briarproject.bramble.api.contact.event.PendingContactRemovedEvent;
import org.briarproject.bramble.api.contact.event.PendingContactStateChangedEvent;
import org.briarproject.bramble.api.db.DatabaseExecutor;
@@ -67,8 +66,7 @@ public class PendingContactListViewModel extends AndroidViewModel
@Override
public void eventOccurred(Event e) {
if (e instanceof PendingContactAddedEvent ||
e instanceof PendingContactStateChangedEvent ||
if (e instanceof PendingContactStateChangedEvent ||
e instanceof PendingContactRemovedEvent) {
loadPendingContacts();
}

View File

@@ -130,16 +130,11 @@ internal class ContactControllerTest : ControllerTest() {
every { contactManager.pendingContacts } returns listOf(
Pair(pendingContact, WAITING_FOR_CONNECTION)
)
every {
ctx.json(
listOf(
JsonDict(
"pendingContact" to pendingContact.output(),
"state" to WAITING_FOR_CONNECTION.output()
)
)
)
} returns ctx
val dict = JsonDict(
"pendingContact" to pendingContact.output(),
"state" to WAITING_FOR_CONNECTION.output()
)
every { ctx.json(listOf(dict)) } returns ctx
controller.listPendingContacts(ctx)
}