From 5f2b5e8b9de5a18aaf4714bb7cf38d8ceeb4ca80 Mon Sep 17 00:00:00 2001 From: m1093782566 Date: Tue, 29 Nov 2016 21:58:18 +0800 Subject: [PATCH] store: check sorted order in TestStoreGetSorted --- store/store_test.go | 27 ++++++++++++++------------- 1 file changed, 14 insertions(+), 13 deletions(-) diff --git a/store/store_test.go b/store/store_test.go index d4b839025..59fcbaf61 100644 --- a/store/store_test.go +++ b/store/store_test.go @@ -127,23 +127,24 @@ func TestStoreGetSorted(t *testing.T) { e, err := s.Get("/foo", true, true) assert.Nil(t, err, "") assert.Equal(t, e.EtcdIndex, eidx, "") + var yNodes NodeExterns - for _, node := range e.Node.Nodes { - switch node.Key { - case "/foo/x": - case "/foo/y": + sortedStrings := []string{"/foo/x", "/foo/y", "/foo/z"} + for i := range e.Node.Nodes { + node := e.Node.Nodes[i] + if node.Key != sortedStrings[i] { + t.Errorf("expect key = %s, got key = %s", sortedStrings[i], node.Key) + } + if node.Key == "/foo/y" { yNodes = node.Nodes - case "/foo/z": - default: - t.Errorf("key = %s, not matched", node.Key) } } - for _, node := range yNodes { - switch node.Key { - case "/foo/y/a": - case "/foo/y/b": - default: - t.Errorf("key = %s, not matched", node.Key) + + sortedStrings = []string{"/foo/y/a", "/foo/y/b"} + for i := range yNodes { + node := yNodes[i] + if node.Key != sortedStrings[i] { + t.Errorf("expect key = %s, got key = %s", sortedStrings[i], node.Key) } } }