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

Unified Diff: appengine/cmd/cron/engine/engine_test.go

Issue 1910633006: LogDog: Support per-namespace expired archival. (Closed) Base URL: https://github.com/luci/luci-go@logdog-coordinator-svcdec
Patch Set: Update another test. Created 4 years, 8 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
« no previous file with comments | « no previous file | appengine/cmd/logdog_coordinator/backend/main.go » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: appengine/cmd/cron/engine/engine_test.go
diff --git a/appengine/cmd/cron/engine/engine_test.go b/appengine/cmd/cron/engine/engine_test.go
index e227e307eb1a9139b64bb3c6afcf371554702b84..0c26d5355cf3c3c2e3504a356daa44f7ed4cbfb0 100644
--- a/appengine/cmd/cron/engine/engine_test.go
+++ b/appengine/cmd/cron/engine/engine_test.go
@@ -619,8 +619,8 @@ func TestPrepareTopic(t *testing.T) {
pubSubCalls := 0
e.configureTopic = func(c context.Context, topic, sub, pushURL, publisher string) error {
pubSubCalls++
- ctx.So(topic, ShouldEqual, "projects/dev~app/topics/dev-cron+noop+some~publisher.com")
- ctx.So(sub, ShouldEqual, "projects/dev~app/subscriptions/dev-cron+noop+some~publisher.com")
+ ctx.So(topic, ShouldEqual, "projects/app/topics/dev-cron+noop+some~publisher.com")
+ ctx.So(sub, ShouldEqual, "projects/app/subscriptions/dev-cron+noop+some~publisher.com")
ctx.So(pushURL, ShouldEqual, "") // pull on dev server
ctx.So(publisher, ShouldEqual, "some@publisher.com")
return nil
@@ -640,7 +640,7 @@ func TestPrepareTopic(t *testing.T) {
// Once.
topic, token, err := ctl.PrepareTopic("some@publisher.com")
So(err, ShouldBeNil)
- So(topic, ShouldEqual, "projects/dev~app/topics/dev-cron+noop+some~publisher.com")
+ So(topic, ShouldEqual, "projects/app/topics/dev-cron+noop+some~publisher.com")
So(token, ShouldNotEqual, "")
So(pubSubCalls, ShouldEqual, 1)
« no previous file with comments | « no previous file | appengine/cmd/logdog_coordinator/backend/main.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698