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

Unified Diff: gm/tests/run.sh

Issue 156023002: gm self-test: stop checking stdout and stderr (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 6 years, 10 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
« no previous file with comments | « gm/tests/outputs/pipe-playback-failure/output-expected/stdout ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: gm/tests/run.sh
diff --git a/gm/tests/run.sh b/gm/tests/run.sh
index 29074e5f1dafccfcb0f9e1e2cd5ef52632930dc8..333f3effc02add713b6fe2acab8797e825adfc6a 100755
--- a/gm/tests/run.sh
+++ b/gm/tests/run.sh
@@ -70,7 +70,6 @@ function assert_fails {
# Run gm...
# - with the arguments in $1
-# - writing stdout into $2/$OUTPUT_ACTUAL_SUBDIR/stdout
# - writing json summary into $2/$OUTPUT_ACTUAL_SUBDIR/json-summary.txt
# - writing return value into $2/$OUTPUT_ACTUAL_SUBDIR/return_value
# Then compare all of those against $2/$OUTPUT_EXPECTED_SUBDIR .
@@ -89,20 +88,9 @@ function gm_test {
COMMAND="$GM_BINARY $GM_ARGS --writeJsonSummaryPath $JSON_SUMMARY_FILE --writePath $ACTUAL_OUTPUT_DIR/writePath --mismatchPath $ACTUAL_OUTPUT_DIR/mismatchPath --missingExpectationsPath $ACTUAL_OUTPUT_DIR/missingExpectationsPath"
- echo "$COMMAND" >$ACTUAL_OUTPUT_DIR/command_line
- $COMMAND >$ACTUAL_OUTPUT_DIR/stdout 2>$ACTUAL_OUTPUT_DIR/stderr
+ $COMMAND
epoger 2014/02/05 18:01:15 This is the only file of interest in the CL. The
echo $? >$ACTUAL_OUTPUT_DIR/return_value
- # Only compare selected lines in the stdout, to ignore any spurious lines
- # as noted in http://code.google.com/p/skia/issues/detail?id=1068 .
- #
- # TODO(epoger): This is still hacky... we need to rewrite this script in
- # Python soon, and make stuff like this more maintainable.
- grep ^GM: $ACTUAL_OUTPUT_DIR/stdout >$ACTUAL_OUTPUT_DIR/stdout-tmp
- mv $ACTUAL_OUTPUT_DIR/stdout-tmp $ACTUAL_OUTPUT_DIR/stdout
- grep ^GM: $ACTUAL_OUTPUT_DIR/stderr >$ACTUAL_OUTPUT_DIR/stderr-tmp
- mv $ACTUAL_OUTPUT_DIR/stderr-tmp $ACTUAL_OUTPUT_DIR/stderr
-
# Replace image file contents with just the filename, for two reasons:
# 1. Image file encoding may vary by platform
# 2. https://code.google.com/p/chromium/issues/detail?id=169600
@@ -279,9 +267,11 @@ done
# Exercise all rebaseline_server unittests.
assert_passes "python gm/rebaseline_server/test_all.py"
+echo
if [ $ENCOUNTERED_ANY_ERRORS == 0 ]; then
echo "All tests passed."
exit 0
else
+ echo "Some tests failed."
exit 1
fi
« no previous file with comments | « gm/tests/outputs/pipe-playback-failure/output-expected/stdout ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698