Bogus tests without test methods are reported as successful tests

It looks good, but the if(TRUE) should be removed before commit.

Sorry, don't know how that happened. Latest patch looks great. I have one minor comment, though:

+++ b/core/modules/simpletest/lib/Drupal/simpletest/TestBase.php
@@ -806,6 +806,9 @@ public function run(array $methods = array()) {
+      $this->assert(FALSE, 'No test methods found.', 'Requirements', array('function' => __METHOD__ . '()', 'file' => __FILE__, 'line' => __LINE__));

A little further down we use TestBase::insertAssert() directly. On the other hand I would have expected $this->fail() instead. Thoughts?

Ahh, OK we use TestBase::insertAssert() so we can get the assertion ID. That is not needed here, so we should really be using $this->fail() as far as I can tell.

That's super minor, though, so let's just get this in.

Oohh, that's interesting. Thanks!

Would have been great to have a comment for that as I can easily see people wanting to "fix" that in the future. But anyway, still RTBC.

Nice optimization.

Committed/pushed to 8.x, with this comment added above the assert:

      // Call $this->assert() here because we need to pass along custom caller
      // information, lest the wrong originating code file/line be identified.

Hopefully that's mostly right. :)

Awesome, thanks! sun++, webchick++ :-)