Issue #2554267 by asgorobets, justAChris, LoMo: Clean Environment button on...

Issue #2554267 by asgorobets, justAChris, LoMo: Clean Environment button on Simpletest UI should not run test even if tests are selected

core/modules/simpletest/src/Form/SimpletestTestForm.php

@@ -62,6 +62,17 @@ public function buildForm(array $form, FormStateInterface $form_state) {
       '#tableselect' => TRUE,
       '#button_type' => 'primary',
     );
+    $form['clean'] = array(
+      '#type' => 'fieldset',
+      '#title' => $this->t('Clean test environment'),
+      '#description' => $this->t('Remove tables with the prefix "simpletest" and temporary directories that are left over from tests that crashed. This is intended for developers when creating tests.'),
+      '#weight' => 200,
+    );
+    $form['clean']['op'] = array(
+      '#type' => 'submit',
+      '#value' => $this->t('Clean environment'),
+      '#submit' => array('simpletest_clean_environment'),
+    );
 
     // Do not needlessly re-execute a full test discovery if the user input
     // already contains an explicit list of test classes to run.
@@ -187,18 +198,6 @@ public function buildForm(array $form, FormStateInterface $form_state) {
       }
     }
 
-    $form['clean'] = array(
-      '#type' => 'fieldset',
-      '#title' => $this->t('Clean test environment'),
-      '#description' => $this->t('Remove tables with the prefix "simpletest" and temporary directories that are left over from tests that crashed. This is intended for developers when creating tests.'),
-      '#weight' => 200,
-    );
-    $form['clean']['op'] = array(
-      '#type' => 'submit',
-      '#value' => $this->t('Clean environment'),
-      '#submit' => array('simpletest_clean_environment'),
-    );
-
     return $form;
   }