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

Side by Side Diff: logdog/client/annotee/annotation/annotation_test.go

Issue 2994593004: [annotee] add SOURCE_MANIFEST annotation. (Closed)
Patch Set: Created 3 years, 4 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
OLDNEW
1 // Copyright 2015 The LUCI Authors. 1 // Copyright 2015 The LUCI Authors.
2 // 2 //
3 // Licensed under the Apache License, Version 2.0 (the "License"); 3 // Licensed under the Apache License, Version 2.0 (the "License");
4 // you may not use this file except in compliance with the License. 4 // you may not use this file except in compliance with the License.
5 // You may obtain a copy of the License at 5 // You may obtain a copy of the License at
6 // 6 //
7 // http://www.apache.org/licenses/LICENSE-2.0 7 // http://www.apache.org/licenses/LICENSE-2.0
8 // 8 //
9 // Unless required by applicable law or agreed to in writing, software 9 // Unless required by applicable law or agreed to in writing, software
10 // distributed under the License is distributed on an "AS IS" BASIS, 10 // distributed under the License is distributed on an "AS IS" BASIS,
(...skipping 261 matching lines...) Expand 10 before | Expand all | Expand 10 after
272 Dir: "/path/to/dir", 272 Dir: "/path/to/dir",
273 Env: map[string]string{ 273 Env: map[string]string{
274 "FOO": "BAR", 274 "FOO": "BAR",
275 "BAZ": "QUX", 275 "BAZ": "QUX",
276 }, 276 },
277 }}, 277 }},
278 {"timestamps", nil}, 278 {"timestamps", nil},
279 {"coverage", nil}, 279 {"coverage", nil},
280 {"nested", nil}, 280 {"nested", nil},
281 {"legacy", nil}, 281 {"legacy", nil},
282 {"source_manifest", nil},
dnj 2017/08/04 23:50:07 Idunno, might as well just slap this onto "coverag
iannucci 2017/08/05 00:27:38 doonnnneeeee
282 } 283 }
283 284
284 if *generate { 285 if *generate {
285 touched := stringset.New(0) 286 touched := stringset.New(0)
286 for _, tc := range testCases { 287 for _, tc := range testCases {
287 if err := tc.generate(t, startTime, touched); err != nil { 288 if err := tc.generate(t, startTime, touched); err != nil {
288 t.Fatalf("Failed to generate %q: %v\n", tc.name, err) 289 t.Fatalf("Failed to generate %q: %v\n", tc.name, err)
289 } 290 }
290 } 291 }
291 292
(...skipping 56 matching lines...) Expand 10 before | Expand all | Expand 10 after
348 349
349 Convey(`Append to a closed State is a no-op.`, func() { 350 Convey(`Append to a closed State is a no-op.`, func() {
350 st := testCases[0].state(startTime) 351 st := testCases[0].state(startTime)
351 st.Finish() 352 st.Finish()
352 sclone := st 353 sclone := st
353 So(st.Append("asdf"), ShouldBeNil) 354 So(st.Append("asdf"), ShouldBeNil)
354 So(st, ShouldResemble, sclone) 355 So(st, ShouldResemble, sclone)
355 }) 356 })
356 }) 357 })
357 } 358 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698