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

Unified Diff: src/trusted/validator/x86/testing/enuminsts/build.scons

Issue 10871078: Enable vdiff test on validator bots (Closed) Base URL: svn://svn.chromium.org/native_client/trunk/src/native_client
Patch Set: Fixing a few nits Created 8 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: src/trusted/validator/x86/testing/enuminsts/build.scons
diff --git a/src/trusted/validator/x86/testing/enuminsts/build.scons b/src/trusted/validator/x86/testing/enuminsts/build.scons
index 00d3bc3c8a4ea81b78f50ed013e0249924b837f8..235c23d5029e15eac2daad0e437bf925cb992220 100644
--- a/src/trusted/validator/x86/testing/enuminsts/build.scons
+++ b/src/trusted/validator/x86/testing/enuminsts/build.scons
@@ -48,5 +48,16 @@ vdiff = test_env.ComponentProgram(
env.NaClTargetArchSuffix('dfa_validate_caller'),
'platform', 'gio', 'utils',
])
-
env.Requires(vdiff, crt)
+
+if env.Bit('target_x86_64'):
+ node = env.CommandTest(
+ 'vdiff.out', command=[vdiff, '--easydiff'],
+ stdout_golden=test_env.File('vdiff_x86_64_baseline.out'),
+ time_warning=800, time_error=2000)
+ env.AddNodeToTestSuite(node, ['validator_diff_tests'], 'vdiff_test')
Karl 2012/08/28 21:23:52 Should there be a blank line here for readability?
Brad Chen 2012/08/28 22:09:42 Done.
+if env.Bit('target_x86_32'):
+ # No golden output for 32-bit until more diffs are fixed
+ node = env.CommandTest('vdiff.out', command=[vdiff],
+ time_warning=800, time_error=2000)
+ env.AddNodeToTestSuite(node, ['validator_diff_tests'], 'vdiff_test')

Powered by Google App Engine
This is Rietveld 408576698