Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(577)

Unified Diff: logdog/appengine/coordinator/hierarchy/component_test.go

Issue 2991253004: [logdog] Remove list functionality. (Closed)
Patch Set: fix test Created 3 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: logdog/appengine/coordinator/hierarchy/component_test.go
diff --git a/logdog/appengine/coordinator/hierarchy/component_test.go b/logdog/appengine/coordinator/hierarchy/component_test.go
deleted file mode 100644
index aee750fdbf9bc8b3946cd59789c8e4f0db928691..0000000000000000000000000000000000000000
--- a/logdog/appengine/coordinator/hierarchy/component_test.go
+++ /dev/null
@@ -1,98 +0,0 @@
-// Copyright 2016 The LUCI Authors.
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-// http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package hierarchy
-
-import (
- "fmt"
- "testing"
-
- "github.com/luci/luci-go/logdog/common/types"
- . "github.com/smartystreets/goconvey/convey"
-)
-
-func TestComponents(t *testing.T) {
- t.Parallel()
-
- tests := []struct {
- path types.StreamPath
- full bool
- comp []*Component
- }{
- {path: "", full: false, comp: nil},
- {path: "foo/bar/+/baz", full: true, comp: []*Component{
- {"foo/bar/+", "baz", true},
- {"foo/bar", "+", false},
- {"foo", "bar", false},
- {"", "foo", false},
- }},
- {path: "foo/bar/+", full: false, comp: []*Component{
- {"foo/bar", "+", false},
- {"foo", "bar", false},
- {"", "foo", false},
- }},
- }
-
- Convey(`Testing MakeComponents`, t, func() {
- for _, tc := range tests {
- Convey(fmt.Sprintf(`Path %q yields %d component(s)`, tc.path, len(tc.comp)), func() {
- So(Components(tc.path, tc.full), ShouldResemble, tc.comp)
- })
- }
- })
-}
-
-func TestComponentID(t *testing.T) {
- t.Parallel()
-
- encodingTestCases := []struct {
- cid componentID
- encoded string
- }{
- {componentID{"foo", "0", false}, "y~8000~foo"},
- {componentID{"foo", "0", true}, "n~8000~foo"},
- {componentID{"foo", "00", true}, "n~8000:8080~foo"},
- {componentID{"foo", "003", true}, "n~81c0:8180~foo"},
- {componentID{"foo", "bar", true}, "s~bar~foo"},
- }
- Convey(`Component ID Encoding`, t, func() {
- for _, tc := range encodingTestCases {
- Convey(fmt.Sprintf(`%+v encodes to %q`, tc.cid, tc.encoded), func() {
- So(tc.cid.key(), ShouldResemble, tc.encoded)
-
- var cid componentID
- cid.setID(tc.encoded)
- So(cid, ShouldResemble, tc.cid)
- })
- }
- })
-
- decodingTestCases := []struct {
- encoded string
- cid componentID
- }{
- {"y~8000:8000~foo", componentID{"foo", "0", false}},
- {"y~8000~foo", componentID{"foo", "0", false}},
- {"n~8000:8080~foo", componentID{"foo", "00", true}},
- }
- Convey(`Component ID Decoding`, t, func() {
- for _, tc := range decodingTestCases {
- Convey(fmt.Sprintf(`%q decodes to %+v`, tc.encoded, tc.cid), func() {
- var cid componentID
- cid.setID(tc.encoded)
- So(cid, ShouldResemble, tc.cid)
- })
- }
- })
-}
« no previous file with comments | « logdog/appengine/coordinator/hierarchy/componentID.go ('k') | logdog/appengine/coordinator/hierarchy/hierarchy.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698