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

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

Issue 9447057: Incremental build is now off by default (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 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
OLDNEW
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, 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 /** 5 /**
6 * Classes and methods for executing tests. 6 * Classes and methods for executing tests.
7 * 7 *
8 * This module includes: 8 * This module includes:
9 * - Managing parallel execution of tests, including timeout checks. 9 * - Managing parallel execution of tests, including timeout checks.
10 * - Evaluating the output of each test as pass/fail/crash/timeout. 10 * - Evaluating the output of each test as pass/fail/crash/timeout.
(...skipping 455 matching lines...) Expand 10 before | Expand all | Expand 10 after
466 _process.close(); 466 _process.close();
467 _startProcess(() { 467 _startProcess(() {
468 _reportResult(">>> TEST TIMEOUT"); 468 _reportResult(">>> TEST TIMEOUT");
469 }); 469 });
470 }; 470 };
471 _process.kill(); 471 _process.kill();
472 }; 472 };
473 } 473 }
474 474
475 void _startProcess(then) { 475 void _startProcess(then) {
476 _process = new Process.start(_executable, ['-batch']); 476 _process = new Process.start(_executable, ['--batch']);
477 _stdoutStream = new StringInputStream(_process.stdout); 477 _stdoutStream = new StringInputStream(_process.stdout);
478 _stderrStream = new StringInputStream(_process.stderr); 478 _stderrStream = new StringInputStream(_process.stderr);
479 _testStdout = new List<String>(); 479 _testStdout = new List<String>();
480 _testStderr = new List<String>(); 480 _testStderr = new List<String>();
481 _stdoutStream.lineHandler = _readOutput(_stdoutStream, _testStdout); 481 _stdoutStream.lineHandler = _readOutput(_stdoutStream, _testStdout);
482 _stderrStream.lineHandler = _readOutput(_stderrStream, _testStderr); 482 _stderrStream.lineHandler = _readOutput(_stderrStream, _testStderr);
483 _process.exitHandler = _exitHandler; 483 _process.exitHandler = _exitHandler;
484 _process.startHandler = then; 484 _process.startHandler = then;
485 } 485 }
486 } 486 }
(...skipping 230 matching lines...) Expand 10 before | Expand all | Expand 10 after
717 // the developer doesn't waste his or her time trying to fix a bunch of 717 // the developer doesn't waste his or her time trying to fix a bunch of
718 // tests that appear to be broken but were actually just flakes that 718 // tests that appear to be broken but were actually just flakes that
719 // didn't get retried because there had already been one failure. 719 // didn't get retried because there had already been one failure.
720 new RunningProcess(test, 720 new RunningProcess(test,
721 _MAX_FAILED_NO_RETRY > _progress.numFailedTests).start(); 721 _MAX_FAILED_NO_RETRY > _progress.numFailedTests).start();
722 } 722 }
723 _numProcesses++; 723 _numProcesses++;
724 } 724 }
725 } 725 }
726 } 726 }
OLDNEW
« no previous file with comments | « compiler/java/com/google/dart/compiler/CommandLineOptions.java ('k') | tools/testing/dart/test_suite.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698