run-tests.sh should allow single test methods to be run

I really like that!

+++ b/core/scripts/run-tests.sh
@@ -625,10 +627,17 @@ function simpletest_script_run_one_test($test_id, $test_class) {
+    $test->run(array_filter([$method]));

This line is really confusing, it would be great to document that.

That is great!

This seems like the kind of issue that'll only make it easier to get D8 done, so looks good to me. Unfortunately I don't believe it's possible to add automated tests for this.

Committed and pushed to 8.0.x. Thanks!

Seems like it could be backported, right?

Backported to Drupal 7. Also cleaned up a minor code issue: function simpletest_script_get_test_list was called twice unnecessarily. The results of the first call were not re-used. In the drupal-8 version of the script this was already fixed.

Please review!

Tests ran successfully this time. Please review!

The patch looks good and seems to work correctly.

The output it prints for "Tests to be run" is a little funny though (i.e. compared to running an entire class worth of tests):

$ php scripts/run-tests.sh --class UserAccountLinksUnitTests::testDisabledAccountLink,UserBlocksUnitTests

Drupal test run
---------------

Tests to be run:
 -  (UserAccountLinksUnitTests::testDisabledAccountLink)
 - User blocks (UserBlocksUnitTests)

Also the behavior is strange if you misspell the method name (especially compared to the existing behavior when you misspell the class name):

$ php scripts/run-tests.sh --class UserAccountLinksUnitTests::testDisabledAccountLinkWRONGNAME

Drupal test run
---------------

Tests to be run:
 -  (UserAccountLinksUnitTests::testDisabledAccountLinkWRONGNAME)

Test run started:
 Thursday, February 11, 2016 - 10:24

Test summary
------------

User account links 0 passes, 0 fails, and 0 exceptions

Test run duration: 0 sec

versus:

$ php scripts/run-tests.sh --class UserAccountLinksUnitTestsWRONGNAME::testDisabledAccountLink
  ERROR: Test class not found: UserAccountLinksUnitTestsWRONGNAME
  Did you mean?
  - UserAccountLinksUnitTests

These are pretty minor issues however - do we know if they are fixed in Drupal 8? (If not, leaving them for a followup would be fine.)

And one small comment about the code:

+        list($class_name, ) = explode('::', $test_class, 2);

Minor, but the comma isn't needed there - it could just be list($class_name).

Besides the above questions I think this should be RTBC.

Drupal 8 doesn't have test names, it always only shows the class name in run-tests.sh and also the UI.

Not sure if what we should do for 7.x.

I have updated the patch with the remarks of #12. Information should be better now for end-users.

--class TestCase::testMethod works but this patch breaks the original --class TestCase.

The patch didn't apply to latest core release. Rerolled.

Fixed the bug causing --class TestClass to fail.

Tested scenarios:
- run-tests.sh ... MyTestGroup: detects all classes, and runs them
- run-tests.sh ... --class MyTestClass: runs only the test methods in the specified class
- run-tests.sh ... --class MyTestClass::testMethod: runs only the specified test method

Had to manually cancel the tests, as this is not covered by tests.

Noticed that earlier patches were checked by the testbot, so decided to re-test #21. Tests are passing, so this should be ready for review.

Hi @jedihe. We used to do back ports in the same issue, but we should create a follow-up issue for your patch in Drupal 7 and then mark this as Fixed and change the version back.

Thanks @mradcliffe! I just filed the separate issue for the D7 backport. Marking this as fixed against 8.0.x-dev (latest 8.x I see used in this issue).