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

Unified Diff: gm/tests/outputs/ignoring-some-failures/output-expected/json-summary.txt

Issue 23799004: gm: add self-test to confirm that ignore-failure field is respected (Closed) Base URL: http://skia.googlecode.com/svn/trunk/
Patch Set: ignore_one_failure Created 7 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 side-by-side diff with in-line comments
Download patch
Index: gm/tests/outputs/ignoring-some-failures/output-expected/json-summary.txt
===================================================================
--- gm/tests/outputs/ignoring-some-failures/output-expected/json-summary.txt (revision 0)
+++ gm/tests/outputs/ignoring-some-failures/output-expected/json-summary.txt (working copy)
@@ -1,10 +1,11 @@
{
"actual-results" : {
"failed" : {
- "565/selftest1.png" : [ "bitmap-64bitMD5", 12927999507540085554 ],
"8888/selftest1.png" : [ "bitmap-64bitMD5", 1209453360120438698 ]
},
- "failure-ignored" : null,
+ "failure-ignored" : {
+ "565/selftest1.png" : [ "bitmap-64bitMD5", 12927999507540085554 ]
epoger 2013/08/30 19:53:02 As expected... we are now ignoring failure for the
+ },
"no-comparison" : null,
"succeeded" : null
},
@@ -13,7 +14,7 @@
"allowed-digests" : [
[ "bitmap-64bitMD5", 8863920166200910451 ]
],
- "ignore-failure" : false
+ "ignore-failure" : true
},
"8888/selftest1.png" : {
"allowed-digests" : [

Powered by Google App Engine
This is Rietveld 408576698