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

Unified Diff: appengine/cmd/dm/mutate/add_finished_deps_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
Index: appengine/cmd/dm/mutate/add_finished_deps_test.go
diff --git a/appengine/cmd/dm/mutate/add_finished_deps_test.go b/appengine/cmd/dm/mutate/add_finished_deps_test.go
index 708f9a4fbf762a159c2007d16b2ad5b867e91bc7..a8e8b280f0afabc4fefb66aa06cfa34766a85e52 100644
--- a/appengine/cmd/dm/mutate/add_finished_deps_test.go
+++ b/appengine/cmd/dm/mutate/add_finished_deps_test.go
@@ -33,7 +33,8 @@ func TestAddFinishedDeps(t *testing.T) {
ID: "to",
BuiltBy: model.TemplateInfo{
*dm.NewTemplateSpec("a", "b", "c", "d"),
- }},
+ },
+ },
},
dm.NewAttemptList(map[string][]uint32{
"to": {1, 2, 3},
@@ -70,7 +71,7 @@ func TestAddFinishedDeps(t *testing.T) {
So(muts, ShouldResemble, []tumble.Mutation{
&AddBackDep{Dep: fs[0].Edge()},
&AddBackDep{Dep: fs[2].Edge()},
- &MergeQuest{f.MergeQuests[0]},
+ &MergeQuest{f.MergeQuests[0], nil},
})
So(ds.GetMulti(fs), ShouldBeNil)
@@ -86,7 +87,7 @@ func TestAddFinishedDeps(t *testing.T) {
Convey("RollForward (bad)", func() {
So(ds.Delete(ak), ShouldBeNil)
_, err := f.RollForward(c)
- So(err, ShouldBeRPCUnauthenticated, "execution Auth")
+ So(err, ShouldBeRPCInternal, "execution Auth")
})
})
}

Powered by Google App Engine
This is Rietveld 408576698