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

Unified Diff: tests/standalone/src/ProcessTestUtil.dart

Issue 8662004: Update test scripts to deal with current VM behavior on optional constructor arguments. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Address review comments Created 9 years, 1 month 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 | « runtime/bin/socket_stream.dart ('k') | tests/standalone/src/TestRunnerTest.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/standalone/src/ProcessTestUtil.dart
diff --git a/tests/standalone/src/ProcessTestUtil.dart b/tests/standalone/src/ProcessTestUtil.dart
index aaa3cf2f386d3a24839ac8e71b014494ff893646..4a465eb170670e24195556cb6772b0c797aef996 100644
--- a/tests/standalone/src/ProcessTestUtil.dart
+++ b/tests/standalone/src/ProcessTestUtil.dart
@@ -3,16 +3,23 @@
// BSD-style license that can be found in the LICENSE file.
String getProcessTestFileName() {
- var names = ['out/Release_ia32/process_test',
- 'out/Debug_ia32/process_test',
- 'xcodebuild/Release_ia32/process_test',
- 'xcodebuild/Debug_ia32/process_test',
- 'Release_ia32/process_test.exe',
- 'Debug_ia32/process_test.exe'];
+ var os = new Platform().operatingSystem();
+
+ var outDir = '';
+ if (os == 'linux') {
+ outDir = 'out';
+ } else if (os == 'macos') {
+ outDir = 'xcodebuild';
+ }
+
+ var names = ['$outDir/Release_ia32/process_test',
+ '$outDir/Debug_ia32/process_test'];
+
for (var name in names) {
if (new File(name).existsSync()) {
return name;
}
}
+
Expect.fail('Could not find the process_test program.');
}
« no previous file with comments | « runtime/bin/socket_stream.dart ('k') | tests/standalone/src/TestRunnerTest.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698