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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: appengine/cmd/dm/mutate/merge_quest_test.go

Issue 1537883002: Initial distributor implementation (Closed) Base URL: https://chromium.googlesource.com/external/github.com/luci/luci-go@master
Patch Set: self review 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
« appengine/cmd/dm/mutate/finish_execution.go ('K') | « appengine/cmd/dm/mutate/merge_quest.go ('k') | appengine/cmd/dm/mutate/record_completion.go » ('j') | appengine/cmd/dm/mutate/schedule_execution.go » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: appengine/cmd/dm/mutate/merge_quest_test.go
diff --git a/appengine/cmd/dm/mutate/merge_quest_test.go b/appengine/cmd/dm/mutate/merge_quest_test.go
index db340a180c7b8c6f1691f40ae3f395e1ee7f1b62..2faa63fb0a3bb6afbb89162c27144621b4fd9505 100644
--- a/appengine/cmd/dm/mutate/merge_quest_test.go
+++ b/appengine/cmd/dm/mutate/merge_quest_test.go
@@ -24,12 +24,12 @@ func TestMergeQuest(t *testing.T) {
c := ttest.Context()
ds := datastore.Get(c)
- desc := dm.NewQuestDesc("distributor", `{"data":"yes"}`)
- qst, err := model.NewQuest(c, desc)
- So(err, ShouldBeNil)
+ desc := dm.NewQuestDesc("distributor", `{"data":"yes"}`, nil)
+ So(desc.Normalize(), ShouldBeNil)
+ qst := model.NewQuest(c, desc)
qst.BuiltBy = append(qst.BuiltBy, *dm.NewTemplateSpec("a", "b", "c", "d"))
- mq := &MergeQuest{qst}
+ mq := &MergeQuest{qst, nil}
Convey("root", func() {
So(mq.Root(c), ShouldResemble, ds.MakeKey("Quest", qst.ID))
« appengine/cmd/dm/mutate/finish_execution.go ('K') | « appengine/cmd/dm/mutate/merge_quest.go ('k') | appengine/cmd/dm/mutate/record_completion.go » ('j') | appengine/cmd/dm/mutate/schedule_execution.go » ('J')

Powered by Google App Engine
This is Rietveld 408576698