feat(BUX-648): change logic for updating PubKey

bitcoin-sv · Mar 29, 2024 · 81d1511 · 81d1511
1 parent 91b596b
commit 81d1511
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/engine/contact_service_test.go b/engine/contact_service_test.go
@@ -276,8 +276,8 @@ func TestClientService_AddContactRequest(t *testing.T) {
 		require.NoError(t, err)
 		require.NotNil(t, contact)
 
-		// mark request as accepted
-		contact.Status = ContactNotConfirmed
+		// mark request as confirmed
+		contact.Status = ContactConfirmed
 		err = contact.Save(ctx)
 		require.NoError(t, err)
 
@@ -295,7 +295,7 @@ func TestClientService_AddContactRequest(t *testing.T) {
 		require.Equal(t, updatedPki, updatedContact.PubKey)
 
 		// status should back to awaiting
-		require.Equal(t, ContactAwaitAccept, updatedContact.Status)
+		require.Equal(t, ContactNotConfirmed, updatedContact.Status)
 	})
 }
 

diff --git a/engine/model_contact.go b/engine/model_contact.go
@@ -129,10 +129,12 @@ func (c *Contact) Confirm() error {
 
 func (c *Contact) UpdatePubKey(pk string) (updated bool) {
 	if c.PubKey != pk {
-		// update and back to awaiting if pub key has changed
-		c.Status = ContactAwaitAccept
 		c.PubKey = pk
 
+		if c.Status == ContactConfirmed {
+			c.Status = ContactNotConfirmed
+		}
+
 		updated = true
 	}