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

Unified Diff: milo/appengine/buildbot/build_test.go

Issue 2525493002: Milo: Add themed page for errors (Closed)
Patch Set: Review Created 4 years 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 | « milo/appengine/buildbot/build.go ('k') | milo/appengine/buildbot/builder.go » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: milo/appengine/buildbot/build_test.go
diff --git a/milo/appengine/buildbot/build_test.go b/milo/appengine/buildbot/build_test.go
index 4575b87e35f8cb093b778387be0c83246931e195..31ae70afd7265c00bf2bbff06812e8da133b69d4 100644
--- a/milo/appengine/buildbot/build_test.go
+++ b/milo/appengine/buildbot/build_test.go
@@ -83,8 +83,7 @@ func TestBuild(t *testing.T) {
Number: 1,
Internal: true,
})
- b, err := getBuild(c, "fake", "fake", 1)
- So(b, ShouldBeNil)
+ _, err := getBuild(c, "fake", "fake", 1)
So(err, ShouldResemble, miloerror.Error{
Message: "Cannot fetch project buildbot-internal:\ndatastore: no such entity",
Code: 500,
« no previous file with comments | « milo/appengine/buildbot/build.go ('k') | milo/appengine/buildbot/builder.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698