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

Side by Side Diff: milo/buildsource/buildbot/console_test.go

Issue 2946443003: Milo: (Breaking proto change) Update console definition (Closed)
Patch Set: ImportFix Created 3 years, 5 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 unified diff | Download patch
« no previous file with comments | « milo/api/resp/console.go ('k') | milo/common/config_test.go » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2017 The LUCI Authors. All rights reserved. 1 // Copyright 2017 The LUCI Authors. All rights reserved.
2 // Use of this source code is governed under the Apache License, Version 2.0 2 // Use of this source code is governed under the Apache License, Version 2.0
3 // that can be found in the LICENSE file. 3 // that can be found in the LICENSE file.
4 4
5 package buildbot 5 package buildbot
6 6
7 import ( 7 import (
8 "context" 8 "context"
9 "strconv" 9 "strconv"
10 "testing" 10 "testing"
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
48 48
49 Convey(`Console tests for buildbot`, t, func() { 49 Convey(`Console tests for buildbot`, t, func() {
50 Convey(`Empty request`, func() { 50 Convey(`Empty request`, func() {
51 cb, err := GetConsoleBuilds(c, []resp.BuilderRef{}, []st ring{}) 51 cb, err := GetConsoleBuilds(c, []resp.BuilderRef{}, []st ring{})
52 So(err, ShouldBeNil) 52 So(err, ShouldBeNil)
53 So(cb, ShouldResemble, [][]*resp.ConsoleBuild{}) 53 So(cb, ShouldResemble, [][]*resp.ConsoleBuild{})
54 }) 54 })
55 Convey(`Basic`, func() { 55 Convey(`Basic`, func() {
56 ref := []resp.BuilderRef{ 56 ref := []resp.BuilderRef{
57 { 57 {
58 » » » » » Module: "buildbot", 58 » » » » » Name: "buildbot/fake/fake",
59 » » » » » Name: "fake/fake",
60 Category: []string{"np"}, 59 Category: []string{"np"},
61 ShortName: "np", 60 ShortName: "np",
62 }, 61 },
63 } 62 }
64 cb, err := GetConsoleBuilds(c, ref, []string{"2", "3", " 5"}) 63 cb, err := GetConsoleBuilds(c, ref, []string{"2", "3", " 5"})
65 So(err, ShouldBeNil) 64 So(err, ShouldBeNil)
66 So(len(cb), ShouldEqual, 3) 65 So(len(cb), ShouldEqual, 3)
67 So(len(cb[0]), ShouldEqual, 1) 66 So(len(cb[0]), ShouldEqual, 1)
68 }) 67 })
69 }) 68 })
70 } 69 }
OLDNEW
« no previous file with comments | « milo/api/resp/console.go ('k') | milo/common/config_test.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698