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

Side by Side Diff: recipe_engine/test_result.proto

Issue 2797023002: Always identify test names that lead to failures (Closed)
Patch Set: proto Created 3 years, 8 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 | « recipe_engine/test.py ('k') | recipe_engine/test_result_pb2.py » ('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 syntax = "proto3"; 5 syntax = "proto3";
6 6
7 package recipe_engine; 7 package recipe_engine;
8 8
9 // Result of running recipe tests. 9 // Result of running recipe tests.
10 message TestResult { 10 message TestResult {
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
44 // Keyword arguments to the function. 44 // Keyword arguments to the function.
45 map<string, string> kwargs = 4; 45 map<string, string> kwargs = 4;
46 46
47 // Filename where the check was added. 47 // Filename where the check was added.
48 string filename = 5; 48 string filename = 5;
49 49
50 // Line number where the check was added. 50 // Line number where the check was added.
51 int64 lineno = 6; 51 int64 lineno = 6;
52 } 52 }
53 53
54 // An internal failure (either ctrl-c interruption, or an exception
55 // while running the test; this could mean a crashing recipe execution,
56 // or even failure in the test runner itself).
57 message InternalFailure {
58 }
59
54 message TestFailure { 60 message TestFailure {
55 oneof failures { 61 oneof failures {
56 DiffFailure diff_failure = 1; 62 DiffFailure diff_failure = 1;
57 63
58 CheckFailure check_failure = 2; 64 CheckFailure check_failure = 2;
65
66 InternalFailure internal_failure = 3;
59 } 67 }
60 } 68 }
61 69
62 message TestFailures { 70 message TestFailures {
63 repeated TestFailure failures = 1; 71 repeated TestFailure failures = 1;
64 } 72 }
65 // Maps test names to failure details. 73 // Maps test names to failure details.
66 map<string, TestFailures> test_failures = 4; 74 map<string, TestFailures> test_failures = 4;
67 75
68 // Module names completely lacking test coverage. 76 // Module names completely lacking test coverage.
69 repeated string uncovered_modules = 5; 77 repeated string uncovered_modules = 5;
70 78
71 // Absolute native paths of unused recipe expectation files. 79 // Absolute native paths of unused recipe expectation files.
72 repeated string unused_expectations = 6; 80 repeated string unused_expectations = 6;
73 } 81 }
OLDNEW
« no previous file with comments | « recipe_engine/test.py ('k') | recipe_engine/test_result_pb2.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698