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

Side by Side Diff: tools/testing/dart/test_options.dart

Issue 145273024: Add support for copying coredumps to /tmp (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « tools/test.dart ('k') | tools/testing/dart/test_progress.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library test_options_parser; 5 library test_options_parser;
6 6
7 import "dart:io"; 7 import "dart:io";
8 import "drt_updater.dart"; 8 import "drt_updater.dart";
9 import "test_suite.dart"; 9 import "test_suite.dart";
10 10
(...skipping 317 matching lines...) Expand 10 before | Expand all | Expand 10 after
328 ), 328 ),
329 new _TestOptionSpecification( 329 new _TestOptionSpecification(
330 'clear_safari_cache', 330 'clear_safari_cache',
331 'Clear the safari cache (i.e., delete it).', 331 'Clear the safari cache (i.e., delete it).',
332 ['--clear_safari_cache'], 332 ['--clear_safari_cache'],
333 [], 333 [],
334 false, 334 false,
335 'bool' 335 'bool'
336 ), 336 ),
337 new _TestOptionSpecification( 337 new _TestOptionSpecification(
338 'copy_coredumps',
339 'If we see a crash that we did not expect, copy the core dumps. '
340 'to /tmp',
341 ['--copy-coredumps'],
342 [],
343 false,
344 'bool'
345 ),
346 new _TestOptionSpecification(
338 'local_ip', 347 'local_ip',
339 'IP address the http servers should listen on.' 348 'IP address the http servers should listen on.'
340 'This address is also used for browsers to connect.', 349 'This address is also used for browsers to connect.',
341 ['--local_ip'], 350 ['--local_ip'],
342 [], 351 [],
343 '127.0.0.1'), 352 '127.0.0.1'),
344 new _TestOptionSpecification( 353 new _TestOptionSpecification(
345 'test_server_port', 354 'test_server_port',
346 'Port for test http server.', 355 'Port for test http server.',
347 ['--test_server_port'], 356 ['--test_server_port'],
(...skipping 261 matching lines...) Expand 10 before | Expand all | Expand 10 after
609 isValid = false; 618 isValid = false;
610 print("Error: shard index is ${config['shard']} out of " 619 print("Error: shard index is ${config['shard']} out of "
611 "${config['shards']} shards"); 620 "${config['shards']} shards");
612 } 621 }
613 622
614 if (config['use_repository_packages'] && config['use_public_packages']) { 623 if (config['use_repository_packages'] && config['use_public_packages']) {
615 isValid = false; 624 isValid = false;
616 print("Cannot have both --use-repository-packages and " 625 print("Cannot have both --use-repository-packages and "
617 "--use-public-packages"); 626 "--use-public-packages");
618 } 627 }
628
629 if (config['copy_coredumps'] && Platform.operatingSystem != 'linux') {
630 isValid = false;
631 print("Coredump copying is not yet available on non linux systems");
632 }
633
619 return isValid; 634 return isValid;
620 } 635 }
621 636
622 /** 637 /**
623 * Recursively expand a configuration with multiple values per key 638 * Recursively expand a configuration with multiple values per key
624 * into a list of configurations with exactly one value per key. 639 * into a list of configurations with exactly one value per key.
625 */ 640 */
626 List<Map> _expandConfigurations(Map configuration) { 641 List<Map> _expandConfigurations(Map configuration) {
627 // Expand the pseudo-values such as 'all'. 642 // Expand the pseudo-values such as 'all'.
628 if (configuration['arch'] == 'all') { 643 if (configuration['arch'] == 'all') {
(...skipping 200 matching lines...) Expand 10 before | Expand all | Expand 10 after
829 return option; 844 return option;
830 } 845 }
831 } 846 }
832 print('Unknown test option $name'); 847 print('Unknown test option $name');
833 exit(1); 848 exit(1);
834 } 849 }
835 850
836 851
837 List<_TestOptionSpecification> _options; 852 List<_TestOptionSpecification> _options;
838 } 853 }
OLDNEW
« no previous file with comments | « tools/test.dart ('k') | tools/testing/dart/test_progress.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698