Minor updates to display of test page, and code tidy.

questiontestrun.php

@@ -203,10 +203,12 @@ if (empty($testscases) && $canedit) {
 }
 
 if (empty($testscases)) {
+    echo "\n<hr/>\n";
     $defaulttest = stack_bulk_tester::create_default_test($question);
     $defaulttestresult = $defaulttest->test_question($questionid, $seed, $context);
     echo stack_string('runquestiontests_explanation');
     echo $defaulttestresult->html_output($question, stack_string('runquestiontests_example'));
+    echo "\n<hr/>\n";
 }
 
 $deployfeedback = optional_param('deployfeedback', null, PARAM_TEXT);

stack/bulktester.class.php

@@ -451,7 +451,7 @@ class stack_bulk_tester {
         }
 
         if (!$tests) {
-            $defaulttest = stack_bulk_tester::create_default_test($question);
+            $defaulttest = self::create_default_test($question);
             $defaulttestresult = $defaulttest->test_question($qid, $seed, $context);
             if ($defaulttestresult->passed()) {
                 $ok = true;

stack/cas/parsingrules/033_no_extra_evaluation.filter.php

@@ -25,8 +25,7 @@ class stack_ast_filter_033_no_extra_evaluation implements stack_cas_astfilter {
     public function filter(MP_Node $ast, array &$errors, array &$answernotes, stack_cas_security $identifierrules): MP_Node {
 
         $process = function($node) use (&$errors, &$answernotes) {
-            if ($node instanceof MP_PrefixOp &&
-                $node->op === "''") {
+            if ($node instanceof MP_PrefixOp && $node->op === "''") {
                 $node->position['invalid'] = true;
                 if (array_search('Illegal_extraevaluation', $answernotes) === false) {
                     $answernotes[] = 'Illegal_extraevaluation';