test_runner: fix missing test diagnostics

Author: MoLow

lib/internal/test_runner/runner.js

@@ -130,9 +130,11 @@ function getRunArgs({path, inspectPort }){
 
 classFileTestextendsTest{
  #buffer =[];
- #handleReportItem({ kind, node, nesting =0}){
+ #handleReportItem({ kind, node,comments,nesting =0}){
 nesting+=1;
-
+if(comments){
+ArrayPrototypeForEach(comments,(comment)=>this.reporter.diagnostic(nesting,this.name,comment));
+}
 switch(kind){
 caseTokenKind.TAP_VERSION:
 // TODO(manekinekko): handle TAP version coming from the parser.

test/message/test_runner_output_cli.out

@@ -57,6 +57,7 @@ TAP version 13
  ---
  duration_ms: *
  ...
+ # this test should pass
  # Subtest: sync throw fail
  not ok 8 - sync throw fail
  ---
@@ -204,6 +205,7 @@ TAP version 13
  *
  *
  ...
+ # this subtest should make its parent test fail
  1..1
  not ok 21 - subtest sync throw fail
  ---
@@ -365,6 +367,7 @@ TAP version 13
  ---
  duration_ms: *
  ...
+ # \#diagnostic
  # Subtest: callback pass
  ok 41 - callback pass
  ---