Issue #2981870 by Lendude, alexpott: Duplicate BrokenSetUpTest for BrowserTestBase

core/modules/simpletest/src/Tests/BrokenSetUpTest.php

@@ -12,7 +12,7 @@
  * database tables. Therefore, we ensure that tests which are not set up
  * properly are skipped.
  *
- * @group simpletest
+ * @group WebTestBase
  * @see \Drupal\simpletest\WebTestBase
  */
 class BrokenSetUpTest extends WebTestBase {

core/tests/Drupal/Tests/Core/Test/BrowserTestBaseTest.php

@@ -79,4 +79,22 @@ public function testGetHttpClientException() {
     $ref_gethttpclient->invoke($btb);
   }
 
+  /**
+   * Test that tearDown doesn't call cleanupEnvironment if setUp is not called.
+   *
+   * @covers ::tearDown
+   */
+  public function testTearDownWithoutSetUp() {
+    $method = 'cleanupEnvironment';
+    $this->assertTrue(method_exists(BrowserTestBase::class, $method));
+    $btb = $this->getMockBuilder(BrowserTestBase::class)
+      ->disableOriginalConstructor()
+      ->setMethods([$method])
+      ->getMockForAbstractClass();
+    $btb->expects($this->never())->method($method);
+    $ref_tearDown = new \ReflectionMethod($btb, 'tearDown');
+    $ref_tearDown->setAccessible(TRUE);
+    $ref_tearDown->invoke($btb);
+  }
+
 }