Skip to content
Snippets Groups Projects
Commit 8591075d authored by Jake Taylor's avatar Jake Taylor :lips:
Browse files

Merge branch 'hotfix/membersMap' into 'master'

Fix TestDatabase

See merge request elixxir/mainnet-commitments!16
parents 411a1abb c5ff4614
Branches master
No related tags found
1 merge request!16Fix TestDatabase
...@@ -35,13 +35,15 @@ func TestDatabase(t *testing.T) { ...@@ -35,13 +35,15 @@ func TestDatabase(t *testing.T) {
t.Fatalf("Failed to setup storage: %+v", err) t.Fatalf("Failed to setup storage: %+v", err)
} }
id1 := id.NewIdFromString("zezimaone", id.Node, t) id1 := id.NewIdFromString("zezimaone", id.Node, t)
memCert1 := []byte("cert1")
memCert2 := []byte("cert2")
m1 := Member{ m1 := Member{
Id: id1.Bytes(), Id: id1.Bytes(),
Cert: []byte("cert1"), Cert: memCert1,
} }
m2 := Member{ m2 := Member{
Id: id.NewIdFromString("zezimatwo", id.Node, t).Bytes(), Id: id.NewIdFromString("zezimatwo", id.Node, t).Bytes(),
Cert: []byte("cert2"), Cert: memCert2,
} }
err = s.InsertMembers([]Member{m1, m2}) err = s.InsertMembers([]Member{m1, m2})
if err != nil { if err != nil {
...@@ -51,7 +53,7 @@ func TestDatabase(t *testing.T) { ...@@ -51,7 +53,7 @@ func TestDatabase(t *testing.T) {
if err != nil { if err != nil {
t.Errorf("Failed to get member: %+v", err) t.Errorf("Failed to get member: %+v", err)
} }
if bytes.Compare(rm.Cert, []byte("cert1")) != 0 { if bytes.Compare(rm.Cert, memCert1) != 0 {
t.Errorf("Members didn't match") t.Errorf("Members didn't match")
} }
err = s.InsertCommitment(Commitment{ err = s.InsertCommitment(Commitment{
......
...@@ -25,7 +25,10 @@ func (db *MapImpl) InsertMembers(members []Member) error { ...@@ -25,7 +25,10 @@ func (db *MapImpl) InsertMembers(members []Member) error {
db.Lock() db.Lock()
defer db.Unlock() defer db.Unlock()
for _, m := range members { for _, m := range members {
db.members[base64.StdEncoding.EncodeToString(m.Id)] = &m db.members[base64.StdEncoding.EncodeToString(m.Id)] = &Member{
Id: m.Id,
Cert: m.Cert,
}
} }
return nil return nil
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment