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

Unified Diff: appengine/logdog/coordinator/endpoints/services/getConfig_test.go

Issue 1910923002: LogDog: Add project namespace to service endpoint. (Closed) Base URL: https://github.com/luci/luci-go@logdog-project-coordinator-backend
Patch Set: Comments. 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
Index: appengine/logdog/coordinator/endpoints/services/getConfig_test.go
diff --git a/appengine/logdog/coordinator/endpoints/services/getConfig_test.go b/appengine/logdog/coordinator/endpoints/services/getConfig_test.go
index 982323e7c3fe8963a251c3f9247bcddb708532e8..97398734be0bee283f882146d59a9834f3f92d6f 100644
--- a/appengine/logdog/coordinator/endpoints/services/getConfig_test.go
+++ b/appengine/logdog/coordinator/endpoints/services/getConfig_test.go
@@ -28,6 +28,9 @@ func TestGetConfig(t *testing.T) {
c := memory.Use(context.Background())
c = settings.Use(c, settings.New(&gaesettings.Storage{}))
+ fs := authtest.FakeState{}
+ c = auth.WithState(c, &fs)
+
svcStub := ct.Services{}
svcStub.InitConfig()
svcStub.ServiceConfig.Coordinator.ServiceAuthGroup = "test-services"
@@ -35,9 +38,6 @@ func TestGetConfig(t *testing.T) {
svr := New()
- fs := authtest.FakeState{}
- c = auth.WithState(c, &fs)
-
Convey(`Returns Forbidden error if not a service.`, func() {
_, err := svr.GetConfig(c, nil)
So(err, ShouldBeRPCPermissionDenied)

Powered by Google App Engine
This is Rietveld 408576698