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

Side by Side Diff: impl/memory/info_test.go

Issue 2062613002: Add TrimmedAppID to info service. (Closed) Base URL: https://chromium.googlesource.com/external/github.com/luci/gae@master
Patch Set: Created 4 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The LUCI Authors. All rights reserved. 1 // Copyright 2015 The LUCI Authors. All rights reserved.
2 // Use of this source code is governed under the Apache License, Version 2.0 2 // Use of this source code is governed under the Apache License, Version 2.0
3 // that can be found in the LICENSE file. 3 // that can be found in the LICENSE file.
4 4
5 package memory 5 package memory
6 6
7 import ( 7 import (
8 "testing" 8 "testing"
9 9
10 "github.com/luci/gae/service/info" 10 "github.com/luci/gae/service/info"
11 "golang.org/x/net/context" 11 "golang.org/x/net/context"
12 12
13 . "github.com/smartystreets/goconvey/convey" 13 . "github.com/smartystreets/goconvey/convey"
14 ) 14 )
15 15
16 func TestMustNamespace(t *testing.T) { 16 func TestMustNamespace(t *testing.T) {
17 Convey("MustNamespace panics", t, func() {
18 c := Use(context.Background())
19 i := info.Get(c)
20
21 So(func() {
22 i.MustNamespace("valid_namespace_name")
23 }, ShouldNotPanic)
24 So(func() {
25 i.MustNamespace("invalid namespace name")
26 }, ShouldPanic)
27 })
28
29 Convey("Testable interface works", t, func() { 17 Convey("Testable interface works", t, func() {
30 c := UseWithAppID(context.Background(), "dev~app-id") 18 c := UseWithAppID(context.Background(), "dev~app-id")
31 19
32 // Default value. 20 // Default value.
33 So(info.Get(c).AppID(), ShouldEqual, "app-id") 21 So(info.Get(c).AppID(), ShouldEqual, "app-id")
34 So(info.Get(c).FullyQualifiedAppID(), ShouldEqual, "dev~app-id") 22 So(info.Get(c).FullyQualifiedAppID(), ShouldEqual, "dev~app-id")
35 So(info.Get(c).RequestID(), ShouldEqual, "test-request-id") 23 So(info.Get(c).RequestID(), ShouldEqual, "test-request-id")
36 24
37 // Setting to "override" applies to initial context. 25 // Setting to "override" applies to initial context.
38 c = info.Get(c).Testable().SetRequestID("override") 26 c = info.Get(c).Testable().SetRequestID("override")
39 So(info.Get(c).RequestID(), ShouldEqual, "override") 27 So(info.Get(c).RequestID(), ShouldEqual, "override")
40 28
41 // Derive inner context, "override" applies. 29 // Derive inner context, "override" applies.
42 c = info.Get(c).MustNamespace("valid_namespace_name") 30 c = info.Get(c).MustNamespace("valid_namespace_name")
43 So(info.Get(c).RequestID(), ShouldEqual, "override") 31 So(info.Get(c).RequestID(), ShouldEqual, "override")
44 }) 32 })
45 } 33 }
OLDNEW
« no previous file with comments | « impl/memory/info.go ('k') | impl/prod/info.go » ('j') | service/info/wrapper.go » ('J')

Powered by Google App Engine
This is Rietveld 408576698