Add more keys to GetSize test

p2p/peerManager/peerdb/datastore_test.go

@@ -3,6 +3,7 @@ package peerdb
 import (
 	"context"
 	"encoding/json"
+	"sync"
 	"testing"
 
 	datastore "github.com/ipfs/go-datastore"
@@ -113,23 +114,91 @@ func TestQuery(t *testing.T) {
 
 func TestGetSize(t *testing.T) {
 	ps, teardown := setupDB(t)
-	t.Cleanup(teardown)
+	defer teardown()
 
-	peer := createPeers(t, 1)[0]
+	peers := createPeers(t, 5)
 
-	key := datastore.NewKey(peer.AddrInfo.ID.String())
+	keys := make([]datastore.Key, len(peers))
+
+	cases := []struct {
+		i       int
+		Entropy uint64
+		PubKey  []byte
+	}{
+		{
+			i:       0,
+			Entropy: uint64(12345),
+			PubKey:  []byte(""),
+		},
+		{
+			i:       1,
+			Entropy: uint64(1234567890),
+			PubKey:  []byte("pub"),
+		},
+		{
+			i:       2,
+			Entropy: uint64(1),
+			PubKey:  []byte("pubkey"),
+		},
+		{
+			i:       3,
+			Entropy: uint64(0),
+			PubKey:  []byte("pubkey1234567890"),
+		},
+		{
+			i:       4,
+			Entropy: uint64(12345678901234),
+			PubKey:  []byte("pubkey12345678901234567890"),
+		},
+	}
+
+	var wg sync.WaitGroup
+
+	// Add value to keys and test first time
+	for i, peer := range peers {
+		wg.Add(1)
+		go func(peer *PeerInfo, i int) {
+			defer wg.Done()
+			keys[i] = datastore.NewKey(peer.AddrInfo.ID.String())
+			value, err := json.Marshal(peer)
+			require.NoError(t, err)
+
+			err = ps.Put(context.Background(), keys[i], value)
+			require.NoError(t, err)
+			size, err := ps.GetSize(context.Background(), keys[i])
+			require.NoError(t, err)
+			require.Equal(t, len(value), size)
+		}(peer, i)
+	}
+	wg.Wait()
+
+	// Update keys is parallel and check if size is updated
+	for i, peer := range peers {
+		wg.Add(1)
+		go func(peer *PeerInfo, i int) {
+			defer wg.Done()
+			peer.Entropy = cases[i].Entropy
+			testSize(t, ps, peer, keys[i])
+
+			peer.PubKey = cases[i].PubKey
+			testSize(t, ps, peer, keys[i])
+		}(peer, i)
+	}
+	wg.Wait()
+
+	//Test with non existent key
+	size, err := ps.GetSize(context.Background(), datastore.NewKey("non-existent-key"))
+	require.Error(t, err)
+	require.Equal(t, 0, size)
+}
+
+func testSize(t *testing.T, ps *PeerDB, peer *PeerInfo, key datastore.Key) {
 	value, err := json.Marshal(peer)
 	require.NoError(t, err)
 
 	err = ps.Put(context.Background(), key, value)
 	require.NoError(t, err)
-
 	size, err := ps.GetSize(context.Background(), key)
 	require.NoError(t, err)
 	require.Equal(t, len(value), size)
-
-	//Test with non existent key
-	size, err = ps.GetSize(context.Background(), datastore.NewKey("non-existent-key"))
-	require.Error(t, err)
-	require.Equal(t, 0, size)
 }

p2p/peerManager/peerdb/main_test.go

@@ -6,6 +6,7 @@ import (
 	"testing"
 
 	"github.com/dominant-strategies/go-quai/cmd/utils"
+	"github.com/dominant-strategies/go-quai/common"
 	"github.com/libp2p/go-libp2p/core/crypto"
 	"github.com/libp2p/go-libp2p/core/peer"
 	"github.com/multiformats/go-multiaddr"
@@ -25,7 +26,8 @@ func setupDB(t *testing.T) (*PeerDB, func()) {
 	t.Helper()
 	viper.GetViper().Set(utils.DataDirFlag.Name, os.TempDir())
 	dbDir := "testdb"
-	locationName := "zone-0-0"
+	location := common.Location{0, 0}
+	locationName := location.Name()
 
 	// creat a new peerdb
 	ps, err := NewPeerDB(dbDir, locationName)