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

Merge branch 'xx-4171/fix-map-search' into 'release'

Search in map backend wasn't returning username

See merge request elixxir/user-discovery-bot!60
parents 2993c527 0c92f65c
No related branches found
No related tags found
2 merge requests!68Release,!60Search in map backend wasn't returning username
......@@ -151,6 +151,7 @@ func (m *MapImpl) Search(factHashes [][]byte) ([]*User, error) {
return nil, errors.New("no user associated with hash, this should not be possible")
}
users[*uid] = &User{
Username: u.Username,
Id: u.Id,
RsaPub: u.RsaPub,
DhPub: u.DhPub,
......
......@@ -412,6 +412,7 @@ func TestMapImpl_Search(t *testing.T) {
uid := id.NewIdFromString("testuserid", id.User, t)
user := &User{
Id: uid.Marshal(),
Username: "zezima",
RsaPub: "testrsa",
DhPub: []byte("testdh"),
Salt: []byte("testsalt"),
......@@ -442,4 +443,7 @@ func TestMapImpl_Search(t *testing.T) {
if len(ulist) != 1 {
t.Errorf("Did not receive expected num users. Received: %d, expected: %d", len(ulist), 1)
}
if user.Username != ulist[0].Username {
t.Errorf("Search did not return expected username\n\tExpected: %s\n\tReceived: %s\n", user.Username, ulist[0].Username)
}
}
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment