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

Unified Diff: appengine/cmd/dm/deps/ensure_attempt_test.go

Issue 1537883002: Initial distributor implementation (Closed) Base URL: https://chromium.googlesource.com/external/github.com/luci/luci-go@master
Patch Set: fix imports and make dummy.go a real file 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « appengine/cmd/dm/deps/common_test.go ('k') | appengine/cmd/dm/deps/ensure_graph_data.go » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: appengine/cmd/dm/deps/ensure_attempt_test.go
diff --git a/appengine/cmd/dm/deps/ensure_attempt_test.go b/appengine/cmd/dm/deps/ensure_attempt_test.go
index 11ddc1aab4647e889c1e761b9bd19ad6acc21e08..2ad1bb6a00e554931b2e900582b01493bf9dd2ee 100644
--- a/appengine/cmd/dm/deps/ensure_attempt_test.go
+++ b/appengine/cmd/dm/deps/ensure_attempt_test.go
@@ -9,7 +9,6 @@ import (
"github.com/luci/gae/service/datastore"
"github.com/luci/luci-go/appengine/cmd/dm/model"
- "github.com/luci/luci-go/appengine/tumble"
"github.com/luci/luci-go/common/api/dm/service/v1"
. "github.com/luci/luci-go/common/testing/assertions"
. "github.com/smartystreets/goconvey/convey"
@@ -19,10 +18,8 @@ func TestEnsureAttempt(t *testing.T) {
t.Parallel()
Convey("EnsureGraphData (Ensuring attempts)", t, func() {
- ttest := &tumble.Testing{}
- c := ttest.Context()
+ ttest, c, _, s := testSetup()
ds := datastore.Get(c)
- s := newDecoratedDeps()
Convey("bad", func() {
Convey("no quest", func() {
@@ -34,7 +31,7 @@ func TestEnsureAttempt(t *testing.T) {
})
Convey("mismatched quest", func() {
_, err := s.EnsureGraphData(c, &dm.EnsureGraphDataReq{
- Quest: []*dm.Quest_Desc{dm.NewQuestDesc("foof", "{}")},
+ Quest: []*dm.Quest_Desc{dm.NewQuestDesc("fakeDistributor", "{}", nil)},
Attempts: dm.NewAttemptList(map[string][]uint32{"quest": {1}}),
})
So(err, ShouldErrLike, "must have a matching Attempts entry")
@@ -42,9 +39,9 @@ func TestEnsureAttempt(t *testing.T) {
})
Convey("good", func() {
- desc := dm.NewQuestDesc("swarming", `{"hi": "there"}`)
- q, err := model.NewQuest(c, desc)
- So(err, ShouldBeNil)
+ desc := dm.NewQuestDesc("fakeDistributor", `{"hi": "there"}`, nil)
+ So(desc.Normalize(), ShouldBeNil)
+ q := model.NewQuest(c, desc)
rsp, err := s.EnsureGraphData(c, &dm.EnsureGraphDataReq{
Quest: []*dm.Quest_Desc{desc},
Attempts: dm.NewAttemptList(map[string][]uint32{q.ID: {1}}),
« no previous file with comments | « appengine/cmd/dm/deps/common_test.go ('k') | appengine/cmd/dm/deps/ensure_graph_data.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698