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

Unified Diff: logdog/client/butlerlib/bootstrap/bootstrap_test.go

Issue 2456673003: Butler: export Coordinator host in environment. (Closed)
Patch Set: Created 4 years, 2 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/client/butlerlib/bootstrap/bootstrap_test.go
diff --git a/logdog/client/butlerlib/bootstrap/bootstrap_test.go b/logdog/client/butlerlib/bootstrap/bootstrap_test.go
index 3a989c0bd3768fe02ba6efc7272ee1edee4923f1..4c823dd0377c05a67b1326d39fa242caf609a19a 100644
--- a/logdog/client/butlerlib/bootstrap/bootstrap_test.go
+++ b/logdog/client/butlerlib/bootstrap/bootstrap_test.go
@@ -53,17 +53,19 @@ func TestBootstrap(t *testing.T) {
})
})
- Convey(`And a stream server path`, func() {
+ Convey(`And the remaining environment parameters`, func() {
env[EnvStreamServerPath] = "test:client:params"
+ env[EnvCoordinatorHost] = "example.appspot.com"
- Convey(`Yields a Bootstrap with a Prefix and Client.`, func() {
+ Convey(`Yields a fully-populated Bootstrap.`, func() {
bs, err := getFromEnv(env, reg)
So(err, ShouldBeNil)
So(bs, ShouldResemble, &Bootstrap{
- Project: "test-project",
- Prefix: "butler/prefix",
- Client: &sentinelClient{},
+ CoordinatorHost: "example.appspot.com",
+ Project: "test-project",
+ Prefix: "butler/prefix",
+ Client: &sentinelClient{},
})
So(regSpec, ShouldEqual, "client:params")
})

Powered by Google App Engine
This is Rietveld 408576698