Added json result of test output to output debug directory.

tools/testing/dart/options.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.
 
 import 'dart:io';
 
 import 'configuration.dart';
 import 'path.dart';
 import 'runtime_updater.dart';
 import 'utils.dart';
@@ skipping 215 matching lines @@
     new _Option.bool('use_sdk', '''Use compiler or runtime from the SDK.
 
 Normally, the compiler or runtimes in PRODUCT_DIR is tested, with
 this option, the compiler or runtime in PRODUCT_DIR/dart-sdk/bin
 is tested.
 
 (Note: currently only implemented for dart2js.)'''),
     // TODO(rnystrom): This does not appear to be used. Remove?
     new _Option('build_directory',
         'The name of the build directory, where products are placed.'),
+    new _Option('debug_output_directory',
+        'The name of the output directory for storing debug and log files.',

[Bill Hesse, 2017/09/01 11:56:13]

for storing log files.  Does any other output actually go there? screenshots?
core dumps?

[mkroghj, 2017/09/04 10:43:34]

Done.

+        defaultsTo: "debug"),

[Bill Hesse, 2017/09/01 11:56:13]

default "logs"?

[mkroghj, 2017/09/04 10:43:34]

Done.

     new _Option.bool('noBatch', 'Do not run tests in batch mode.', 'n'),
     new _Option.bool('dart2js_batch', 'Run dart2js tests in batch mode.'),
     new _Option.bool(
         'append_logs', 'Do not delete old logs but rather append to them.'),
     new _Option.bool('write_debug_log',
         'Don\'t write debug messages to stdout but rather to a logfile.'),
     new _Option.bool('write_test_outcome_log',
         'Write test outcomes to a "${TestUtils.testOutcomeFileName}" file.'),
+    new _Option.bool(
+        'write_result_json_log',
+        'Write test results to a "${TestUtils.testResultFileName}" json file '

[Bill Hesse, 2017/09/01 11:56:13]

.resultLogFileName

[mkroghj, 2017/09/04 10:43:34]

Done.

+        'located at the debug_output_directory.'),
     new _Option.bool('reset_browser_configuration',
         '''Browser specific reset of configuration.
 
 Warning: Using this option may remove your bookmarks and other
 settings.'''),
     new _Option.bool('copy_coredumps',
         '''If we see a crash that we did not expect, copy the core dumps to
 "/tmp".'''),
     new _Option(
         'local_ip',
@@ skipping 29 matching lines @@
 false positves and negatives, but can be useful for quick and
 dirty offline testing when not making changes that affect the
 compiler.''')
   ];
 
   /// For printing out reproducing command lines, we don't want to add these
   /// options.
   static final _blacklistedOptions = [
     'append_logs',
     'build_directory',
+    'debug_output_directory',
     'chrome',
     'copy_coredumps',
     'dart',
     'flutter',
     'drt',
     'exclude_suite',
     'firefox',
     'local_ip',
     'progress',
     'report',
     'safari',
     'shard',
     'shards',
     'step_name',
     'tasks',
     'time',
     'verbose',
     'write_debug_log',
     'write_test_outcome_log',
+    'write_result_json_log'
   ].toSet();
 
   /// Parses a list of strings as test options.
   ///
   /// Returns a list of configurations in which to run the tests.
   /// Configurations are maps mapping from option keys to values. When
   /// encountering the first non-option string, the rest of the arguments are
   /// stored in the returned Map under the 'rest' key.
   List<Configuration> parse(List<String> arguments) {
     var configuration = <String, dynamic>{};
@@ skipping 271 matching lines @@
                 skipCompilation: data["skip_compilation"] as bool,
                 useBlobs: data["use_blobs"] as bool,
                 useSdk: data["use_sdk"] as bool,
                 useFastStartup: data["fast_startup"] as bool,
                 useEnableAsserts: data["enable_asserts"] as bool,
                 useDart2JSWithKernel: data["dart2js_with_kernel"] as bool,
                 useDart2JSWithKernelInSsa:
                     data["dart2js_with_kernel_in_ssa"] as bool,
                 writeDebugLog: data["write_debug_log"] as bool,
                 writeTestOutcomeLog: data["write_test_outcome_log"] as bool,
+                writeResultJsonLog: data["write_result_json_log"] as bool,
                 drtPath: data["drt"] as String,
                 chromePath: data["chrome"] as String,
                 safariPath: data["safari"] as String,
                 firefoxPath: data["firefox"] as String,
                 dartPath: data["dart"] as String,
                 dartPrecompiledPath: data["dart_precompiled"] as String,
                 flutterPath: data["flutter"] as String,
                 taskCount: data["tasks"] as int,
                 timeout: data["timeout"] as int,
                 shardCount: data["shards"] as int,
                 shard: data["shard"] as int,
                 stepName: data["step_name"] as String,
                 testServerPort: data["test_server_port"] as int,
                 testServerCrossOriginPort:
                     data['test_server_cross_origin_port'] as int,
                 testDriverErrorPort: data["test_driver_error_port"] as int,
                 localIP: data["local_ip"] as String,
                 dart2jsOptions: dart2jsOptions,
                 vmOptions: vmOptions,
                 packages: data["packages"] as String,
                 packageRoot: data["package_root"] as String,
                 suiteDirectory: data["suite_dir"] as String,
                 builderTag: data["builder_tag"] as String,
+                debugOutputDirectory: data["debug_output_directory"] as String,
                 reproducingArguments: _reproducingCommand(data));
 
             if (configuration.validate()) {
               result.add(configuration);
             }
           }
         }
       }
     }
 
@@ skipping 120 matching lines @@
       if (name == option.name) return option;
 
       // Allow hyphens instead of underscores as the separator since they are
       // more common for command line flags.
       if (name == option.name.replaceAll("_", "-")) return option;
     }
 
     return null;
   }
 }