Backport from D8: run-tests.sh should allow single test methods to be run

Core maintainers - what requirements do you have in order to get this committed? Thank you.

It'd be amazing to be able to run specific methods from classes! Definitely +1

However, it looks like the patch needs a reroll?

Thank you; I think this looks good.

I've done a bit of manual testing and it all seems to work as expected.

There are a couple of quirks like this:

Tests to be run:
 - User login (UserLoginTestCase::testPerUserLoginFloodControl)
 - User login (UserLoginTestCase::testLoginWithAnonSession)

...snip...

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

User login 64 passes, 0 fails, 1 exception, and 20 debug messages
User login 10 passes, 0 fails, 0 exceptions, and 3 debug messages

...where ideally the summary would show which results correspond to each specific method.

However, I don't think that should block us getting this in. More details are available from the verbose output.

I forgot to mention that there's a whitespace issue in the comment at the start of the file (is it a tab being introduced?) in the patch.

That can be fixed on commit.

+++ b/scripts/run-tests.sh
@@ -598,7 +617,12 @@ function simpletest_script_reporter_init() {
-      $info = call_user_func(array($class_name, 'getInfo'));
+      if (strpos($class_name, '::') > 0) {
+        list($test_class, $method) = explode('::', $class_name, 2);
+        $info = call_user_func(array($test_class, 'getInfo'));
+      } else {
+        $info = call_user_func(array($class_name, 'getInfo'));
+      }

Let's use a different variable name at least, it's confusing to have first test_class and then class_name and then the other way round.

Maybe as it's our loop just rename class_name to test_class first and then use the usual variable name.

It's a nit, but it confused me in code review.

Besides that: RTBC + 1

Small tweak to the variable names as they were a bit confusing.

Adding credit from original issue per #2.

Thank you!

Thanks for finishing it off and the quick commit, mcdruid, and Fabianx thanks for the review!