Add support for copying coredumps to /tmp

tools/testing/dart/test_progress.dart

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library test_progress;
 
 import "dart:async";
 import "dart:io";
 import "dart:io" as io;
 import "dart:convert" show JSON;
@@ skipping 258 matching lines @@
 
   void _writeTestOutcomeRecord(Map record) {
     if (_sink == null) {
       _sink = new File(TestUtils.testOutcomeFileName())
           .openWrite(mode: FileMode.APPEND);
     }
     _sink.write("${JSON.encode(record)}\n");
   }
 }
 
+
+class UnexpectedCrashDumpArchiver extends EventListener {
+  void done(TestCase test) {
+    if (test.unexpectedOutput && test.result == Expectation.CRASH) {
+      var name = "core.dart.${test.lastCommandOutput.pid}";
+      var file = new File(name);
+      if (file.existsSync()) {
+        // Find the binary - we assume this is the first part of the command
+        var binName = test.lastCommandExecuted.toString().split(' ').first;
+        var binFile = new File(binName);
+        var binBaseName = new Path(binName).filename;
+        if (file.existsSync()) {

[Bill Hesse, 2014/01/29 07:01:52]

I think this is always true, since it was true above.  Do you mean if
binFile.existsSync?

[ricow1, 2014/01/29 09:41:38]

yes, thank you

+          var dir = new Path(TestUtils.mkdirRecursive(new Path('/tmp'),

[Bill Hesse, 2014/01/29 07:01:52]

We don't want to use Directory.systemTemp for this?

[ricow1, 2014/01/29 09:41:38]

Done.

+              new Path('coredump_${test.lastCommandOutput.pid}')).path);
+          TestUtils.copyFile(new Path(name), dir.append(name));
+          TestUtils.copyFile(new Path(binName), dir.append(binBaseName));

[Bill Hesse, 2014/01/29 07:01:52]

I hate to see all these new uses of Path go in, but they will be easy to change
whenever we decide to get rid of Path.

[ricow1, 2014/01/29 09:41:38]

Yes I thought about that, but our copy functions take paths so I left it for
consistency

+          print("\nCopied core dump and binary for unexpected crash to: "
+                "$dir");
+        }
+      }
+    }
+  }
+}
+
+
 class SummaryPrinter extends EventListener {
   void allTestsKnown() {
     if (SummaryReport.total > 0) {
       SummaryReport.printReport();
     }
   }
 }
 

[Bill Hesse, 2014/01/29 07:01:52]

Add a line here?

[ricow1, 2014/01/29 09:41:38]

Done.

 class TimingPrinter extends EventListener {
   final _command2testCases = new Map<Command, List<TestCase>>();
   final _commandOutputs = new Set<CommandOutput>();
   DateTime _startTime;
 
   TimingPrinter(this._startTime);
 
   void done(TestCase testCase) {
     for (var commandOutput in testCase.commandOutputs.values) {
       var command = commandOutput.command;
@@ skipping 152 matching lines @@
 class LineProgressIndicator extends EventListener {
   void done(TestCase test) {
     var status = 'pass';
     if (test.unexpectedOutput) {
       status = 'fail';
     }
     print('Done ${test.configurationString} ${test.displayName}: $status');
   }
 }
 
+
 class TestFailurePrinter extends EventListener {
   bool _printSummary;
   var _formatter;
   var _failureSummary = <String>[];
   var _failedTests= 0;
 
   TestFailurePrinter(this._printSummary,
                      [this._formatter = const Formatter()]);
 
   void done(TestCase test) {
@@ skipping 161 matching lines @@
       return new VerboseProgressIndicator(startTime);
     case 'status':
       return new ProgressIndicator(startTime);
     case 'buildbot':
       return new BuildbotProgressIndicator(startTime);
     default:
       assert(false);
       break;
   }
 }