Skip to content

Commit 48c10d2

Browse files
Check sort in a different way
1 parent 591dd73 commit 48c10d2

1 file changed

Lines changed: 10 additions & 5 deletions

File tree

internal/signer/sort_test.go

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ package signer_test
22

33
import (
44
"encoding/json"
5-
"reflect"
65
"sort"
76
"testing"
87

@@ -61,11 +60,17 @@ func TestSignerSortByIDs(t *testing.T) {
6160

6261
sort.Sort(signer.ByIDs(s))
6362

64-
sorted := []keys.Signer{
65-
s4, s1, s5, s2, s3,
63+
signerIDs := []string{}
64+
65+
for i, signer := range s {
66+
ids := signer.PublicData().IDs()
67+
if len(ids) != 1 {
68+
t.Errorf("Signer %v IDs %v should have length 1", i, ids)
69+
}
70+
signerIDs = append(signerIDs, ids[0])
6671
}
6772

68-
if !reflect.DeepEqual(s, sorted) {
69-
t.Errorf("Signers incorrectly sorted: got %+v, want %+v", s, sorted)
73+
if !sort.StringsAreSorted(signerIDs) {
74+
t.Errorf("Signers incorrectly sorted: %+v", signerIDs)
7075
}
7176
}

0 commit comments

Comments
 (0)