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

Unified Diff: tool/test.sh

Issue 1188173003: have build/test scripts automatically update baselines (Closed) Base URL: git@github.com:dart-lang/dev_compiler.git@master
Patch Set: Created 5 years, 6 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 | « tool/build_sdk.sh ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tool/test.sh
diff --git a/tool/test.sh b/tool/test.sh
index 09523247ded2e5e89762f3e04460974e7fb98316..ae4590bfa2c8508bbaf9067809a77af5554cad9a 100755
--- a/tool/test.sh
+++ b/tool/test.sh
@@ -6,24 +6,6 @@ function fail {
return 1
}
-# Arguments passed to the diff tool. We exclude:
-# - *.map files so they aren't compared, as the diff is not human readable.
-# - runtime JS files that are just copied over from the sources and are not
-# duplicated in the expected folder.
-DIFF_ARGS="-u -r -N --exclude=\*.map expect actual"
-
-function show_diff {
- echo "Fail: actual output did not match expected"
- echo
- diff $DIFF_ARGS |\
- sed -e "s/^\(+.*\)/\1/" |\
- sed -e "s/^\(-.*\)/\1/"
- echo
- echo "You can update these expectations with:"
- echo "$ pushd `pwd` && cp -a actual/* expect && popd"
- fail
-}
-
# Some tests require being run from the package root
# switch to the root directory of dev_compiler
cd $( dirname "${BASH_SOURCE[0]}" )/..
@@ -40,12 +22,6 @@ cd $( dirname "${BASH_SOURCE[0]}" )/..
unset COVERALLS_TOKEN
pub run test:test test/all_tests.dart || fail
-# validate codegen_test output
-pushd test/codegen/ &> /dev/null
-rm -r actual/dev_compiler/ actual/sunflower/dev_compiler
-diff $DIFF_ARGS > /dev/null || show_diff
-popd &> /dev/null
-
# run self host and analyzer after other tests, because they're ~seconds to run.
pub run test:test test/checker/self_host_test.dart || fail
« no previous file with comments | « tool/build_sdk.sh ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698