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

Unified Diff: tokenserver/appengine/delegation/rpc_import_delegation_configs_test.go

Issue 2413683004: token-server: Delegation config import, validation and evaluation. (Closed)
Patch Set: also check validity_duration 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: tokenserver/appengine/delegation/rpc_import_delegation_configs_test.go
diff --git a/tokenserver/appengine/delegation/rpc_import_delegation_configs_test.go b/tokenserver/appengine/delegation/rpc_import_delegation_configs_test.go
new file mode 100644
index 0000000000000000000000000000000000000000..952b02e6966ffa36c8782ab727d34fbd536d257a
--- /dev/null
+++ b/tokenserver/appengine/delegation/rpc_import_delegation_configs_test.go
@@ -0,0 +1,94 @@
+// Copyright 2016 The LUCI Authors. All rights reserved.
+// Use of this source code is governed under the Apache License, Version 2.0
+// that can be found in the LICENSE file.
+
+package delegation
+
+import (
+ "testing"
+
+ "golang.org/x/net/context"
+
+ "github.com/luci/gae/service/info"
+ "github.com/luci/luci-go/appengine/gaetesting"
+ "github.com/luci/luci-go/common/clock/testclock"
+ "github.com/luci/luci-go/common/config"
+ "github.com/luci/luci-go/common/config/impl/memory"
+ admin "github.com/luci/luci-go/tokenserver/api/admin/v1"
+
+ . "github.com/luci/luci-go/common/testing/assertions"
+ . "github.com/smartystreets/goconvey/convey"
+)
+
+func TestImportDelegationConfigs(t *testing.T) {
+ Convey("Works", t, func() {
+ ctx := gaetesting.TestingContext()
+ ctx, _ = testclock.UseTime(ctx, testclock.TestTimeUTC)
+
+ ctx = prepareCfg(ctx, `rules {
+ name: "rule 1"
+ requestor: "user:some-user@example.com"
+ target_service: "service:some-service"
+ allowed_to_impersonate: "group:some-group"
+ allowed_audience: "REQUESTOR"
+ max_validity_duration: 86400
+ }`)
+
+ rpc := ImportDelegationConfigsRPC{}
+
+ // No config.
+ cfg, err := FetchDelegationConfig(ctx)
+ So(err, ShouldBeNil)
+ So(cfg.Config, ShouldBeNil)
+
+ resp, err := rpc.ImportDelegationConfigs(ctx, nil)
+ So(err, ShouldBeNil)
+ So(resp, ShouldResemble, &admin.ImportedConfigs{
+ ImportedConfigs: []*admin.ImportedConfigs_ConfigFile{
+ {
+ Name: "delegation.cfg",
+ Revision: "780529c8f5e6b219e27482978d792d580f7d4f9d",
+ },
+ },
+ })
+
+ // Have config now.
+ cfg, err = FetchDelegationConfig(ctx)
+ So(err, ShouldBeNil)
+ So(cfg.Config, ShouldNotBeNil)
+ So(cfg.Revision, ShouldEqual, "780529c8f5e6b219e27482978d792d580f7d4f9d")
+
+ // Noop import.
+ resp, err = rpc.ImportDelegationConfigs(ctx, nil)
+ So(err, ShouldBeNil)
+ So(resp.ImportedConfigs[0].Revision, ShouldEqual, "780529c8f5e6b219e27482978d792d580f7d4f9d")
+
+ // Try to import completely broken config.
+ ctx = prepareCfg(ctx, `I'm broken`)
+ _, err = rpc.ImportDelegationConfigs(ctx, nil)
+ So(err, ShouldErrLike, `can't parse config file - line 1.0: unknown field name`)
+
+ // Old config is not replaced.
+ cfg, _ = FetchDelegationConfig(ctx)
+ So(cfg.Revision, ShouldEqual, "780529c8f5e6b219e27482978d792d580f7d4f9d")
+
+ // Try to import a config that doesn't pass validation.
+ ctx = prepareCfg(ctx, `rules {
+ name: "rule 1"
+ }`)
+ _, err = rpc.ImportDelegationConfigs(ctx, nil)
+ So(err, ShouldErrLike, `validation error - rule #1 ("rule 1"): 'requestor' is required (and 4 other errors)`)
+
+ // Old config is not replaced.
+ cfg, _ = FetchDelegationConfig(ctx)
+ So(cfg.Revision, ShouldEqual, "780529c8f5e6b219e27482978d792d580f7d4f9d")
+ })
+}
+
+func prepareCfg(c context.Context, configFile string) context.Context {
+ return config.SetImplementation(c, memory.New(map[string]memory.ConfigSet{
+ "services/" + info.AppID(c): {
+ "delegation.cfg": configFile,
+ },
+ }))
+}

Powered by Google App Engine
This is Rietveld 408576698