Add functionality to open results in browser window to run-tests.sh

Two problems: the browser did not opened (haven't debugged it but i guess you should check for shell_exec == 0?).
Sadly I also don't see the test to finish, there seems to be a blocking process.

1. +++ b/core/scripts/run-tests.sh
   @@ -1059,3 +1073,30 @@ function simpletest_script_color_code($status) {
   +function simpletest_script_open_browser() {
   +  global $args, $test_ids;
   

   It would be great to just pass this as parameters ... but yeah not important.

2. +++ b/core/scripts/run-tests.sh
   @@ -1059,3 +1073,30 @@ function simpletest_script_color_code($status) {
   +  else if (shell_exec('which open')) {
   ...
   +  }
   +  else if (substr(PHP_OS, 0, 3) == 'WIN') {
   

   ha, even a core committer does code style mistakes: else if => elseif (yes I do hate this policy)

Rerolled. If you use --browser in this version, you get:

PHP Fatal error:  Uncaught exception 'Symfony\Component\Routing\Exception\RouteNotFoundException' in /Library/WebServer/Documents/drupal8x/core/lib/Drupal/Core/Routing/NullGenerator.php:47
Stack trace:
#0 /Library/WebServer/Documents/drupal8x/core/lib/Drupal/Core/Routing/UrlGenerator.php(151): Drupal\Core\Routing\NullGenerator->getRoute('simpletest.resu...')
#1 /Library/WebServer/Documents/drupal8x/core/lib/Drupal.php(509): Drupal\Core\Routing\UrlGenerator->generateFromRoute('simpletest.resu...', Array, Array)
#2 /Library/WebServer/Documents/drupal8x/core/modules/simpletest/src/Form/SimpletestResultsForm.php(212): Drupal::url('simpletest.resu...', Array)
#3 /Library/WebServer/Documents/drupal8x/core/scripts/run-tests.sh(1193): Drupal\simpletest\Form\SimpletestResultsForm->buildForm(Array, Object(Drupal\Core\Form\FormState), '3')
#4 /Library/WebServer/Documents/drupal8x/core/scripts/run-tests.sh(96): simpletest_script_open_browser()
#5 {main}
  thrown in /Library/WebServer/Documents/drupal8x/core/lib/Drupal/Core/Routing/NullGenerator.php on line 47

Fatal error: Uncaught exception 'Symfony\Component\Routing\Exception\RouteNotFoundException' in /Library/WebServer/Documents/drupal8x/core/lib/Drupal/Core/Routing/NullGenerator.php:47
Stack trace:
#0 /Library/WebServer/Documents/drupal8x/core/lib/Drupal/Core/Routing/UrlGenerator.php(151): Drupal\Core\Routing\NullGenerator->getRoute('simpletest.resu...')
#1 /Library/WebServer/Documents/drupal8x/core/lib/Drupal.php(509): Drupal\Core\Routing\UrlGenerator->generateFromRoute('simpletest.resu...', Array, Array)
#2 /Library/WebServer/Documents/drupal8x/core/modules/simpletest/src/Form/SimpletestResultsForm.php(212): Drupal::url('simpletest.resu...', Array)
#3 /Library/WebServer/Documents/drupal8x/core/scripts/run-tests.sh(1193): Drupal\simpletest\Form\SimpletestResultsForm->buildForm(Array, Object(Drupal\Core\Form\FormState), '3')
#4 /Library/WebServer/Documents/drupal8x/core/scripts/run-tests.sh(96): simpletest_script_open_browser()
#5 {main}
  thrown in /Library/WebServer/Documents/drupal8x/core/lib/Drupal/Core/Routing/NullGenerator.php on line 47

@alexpott suggests adding a static method to SimpletestResultsForm that returns the form parts needed in run-tests.sh and also call it in SimpletestResultsForm::buildForm().

Manually tested the patch, that is really cool, it works for me!

+++ b/core/modules/simpletest/src/Form/SimpletestResultsForm.php
@@ -58,37 +59,37 @@ public function __construct(Connection $database) {
-      '#alt' => $this->t('Pass'),
+      '#alt' => $translation->translate('Pass'),
...
-      '#alt' => $this->t('Fail'),
+      '#alt' => $translation->translate('Fail'),
...
-      '#alt' => $this->t('Exception'),
+      '#alt' => $translation->translate('Exception'),

Mh, isn't the point of this ->t function that potx can scan them? Maybe we need a static wrapper for t?

#9 works for me too. The results styling could be better themed.

It is very plain as compared to the UI form.

Also I just noticed that on Firefox, the disclosure triangle is missing and its function doesn't work but it does work in Chrome. The web console is not really helpful on why.

@alexpott Agreed - fixed!

+++ b/core/scripts/run-tests.sh
@@ -291,6 +305,10 @@ function simpletest_script_parse_args() {
+  //

Pure lonely comment :P

This seems useful to help with debugging. Removed the phantom comment.

Committed and pushed to 8.0.x. Thanks!

Let's backport this.