Revert "test_runner: do not invoke after hook when test is empty"

This reverts commit a53fd95. This caused a regression because the original issue this commit was attempting to fix is not a bug. The after() hook should always run. Fixes: #51997 PR-URL: #51998 Reviewed-By: Matthew Aitken <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>

Author: cjihrig

lib/internal/test_runner/test.js

@@ -595,9 +595,7 @@ class Test extends AsyncResource{
 
 const{ args, ctx }=this.getRunArgs();
 constafter=async()=>{
-// If its a root test then check for global after hook else check for parent after hook
-constcheck=this.parent ? this.parent.hooks.after.length>0 : this.hooks.after.length>0;
-if(check){
+if(this.hooks.after.length>0){
 awaitthis.runHook('after',{__proto__: null, args, ctx });
 }
 };