Unverified Commit 6be8b1d1 authored by Alex Pott's avatar Alex Pott
Browse files

Issue #3113988 by longwave: Remove all @deprecated code in run-tests.sh

parent 6d1348a4
Loading
Loading
Loading
Loading
+0 −35
Changes for core/scripts/run-tests.sh: 0 added lines, 35 removed lines.
Original line number Diff line number Diff line
@@ -177,12 +177,7 @@
TestDatabase::releaseAllTestLocks();

// Display results before database is cleared.
if ($args['browser']) {
  simpletest_script_open_browser();
}
else {
simpletest_script_reporter_display_results();
}

if ($args['xml']) {
  simpletest_script_reporter_write_xml_results();
@@ -317,10 +312,6 @@ function simpletest_script_help() {
              test database and configuration directories. Use in combination
              with --repeat for debugging random test failures.

  --browser   Deprecated, use --verbose instead. This enforces --keep-results and
              if you want to also view any pages rendered in the simpletest
              browser you need to add --verbose to the command line.

  --non-html  Removes escaping from output. Useful for reading results on the
              CLI.

@@ -398,7 +389,6 @@ function simpletest_script_parse_args() {
    'repeat' => 1,
    'die-on-fail' => FALSE,
    'suppress-deprecations' => FALSE,
    'browser' => FALSE,
    // Used internally.
    'test-id' => 0,
    'execute-test' => '',
@@ -449,10 +439,6 @@ function simpletest_script_parse_args() {
    exit(SIMPLETEST_SCRIPT_EXIT_FAILURE);
  }

  if ($args['browser']) {
    simpletest_script_print_error('The --browser option is deprecated in drupal:8.8.0 and is removed from drupal:9.0.0. Use --verbose instead. See https://www.drupal.org/node/3083549');
    $args['keep-results'] = TRUE;
  }
  return [$args, $count];
}

@@ -1522,24 +1508,3 @@ function simpletest_script_load_messages_by_test_id($test_ids) {

  return $results;
}

/**
 * Display test results.
 *
 * @deprecated in drupal:8.8.0 and is removed from drupal:9.0.0. This function
 *   supports the --browser option in this script. Use the --verbose option
 *   instead.
 *
 * @see https://www.drupal.org/node/3083549
 *
 * @todo Remove this in https://www.drupal.org/project/drupal/issues/3075490.
 */
function simpletest_script_open_browser() {
  // Note: the user already has received a message about the deprecation in CLI
  // so we trigger an error just in case this method has been used as API.
  @trigger_error('The --browser option is deprecated in drupal:8.8.0 and is removed from drupal:9.0.0. Use --verbose instead. See https://www.drupal.org/node/3083549', E_USER_DEPRECATED);
  if (function_exists('_simpletest_run_tests_script_open_browser')) {
    return _simpletest_run_tests_script_open_browser();
  }
  simpletest_script_print_error('In order to use the --browser option the Simpletest module must be available. See https://www.drupal.org/node/3083549.');
}