test_runner: don't use a symbol for runHook()

This is not exposed to userland, so there is no need to put it
behind a symbol.
PR-URL: #45792
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

Author: cjihrig

lib/internal/test_runner/test.js

@@ -75,7 +75,6 @@ const testNamePatterns = testNamePatternFlag?.length > 0 ?
 (re)=>convertStringToRegExp(re,'--test-name-pattern')
 ) : null;
 constkShouldAbort=Symbol('kShouldAbort');
-constkRunHook=Symbol('kRunHook');
 constkHookNames=ObjectSeal(['before','after','beforeEach','afterEach']);
 constkUnwrapErrors=newSafeSet()
 .add(kTestCodeFailure).add(kHookFailure)
@@ -476,7 +475,7 @@ class Test extends AsyncResource {
 return{ ctx,args: [ctx]};
 }
 
-async[kRunHook](hook,args){
+asyncrunHook(hook,args){
 validateOneOf(hook,'hook name',kHookNames);
 try{
 awaitArrayPrototypeReduce(this.hooks[hook],async(prev,hook)=>{
@@ -507,13 +506,13 @@ class Test extends AsyncResource {
 const{ args, ctx }=this.getRunArgs();
 constafterEach=runOnce(async()=>{
 if(this.parent?.hooks.afterEach.length>0){
-awaitthis.parent[kRunHook]('afterEach',{ args, ctx });
+awaitthis.parent.runHook('afterEach',{ args, ctx });
 }
 });
 
 try{
 if(this.parent?.hooks.beforeEach.length>0){
-awaitthis.parent[kRunHook]('beforeEach',{ args, ctx });
+awaitthis.parent.runHook('beforeEach',{ args, ctx });
 }
 conststopPromise=stopTest(this.timeout,this.signal);
 construnArgs=ArrayPrototypeSlice(args);
@@ -761,9 +760,10 @@ class Suite extends Test {
 consthookArgs=this.getRunArgs();
 constafterEach=runOnce(async()=>{
 if(this.parent?.hooks.afterEach.length>0){
-awaitthis.parent[kRunHook]('afterEach',hookArgs);
+awaitthis.parent.runHook('afterEach',hookArgs);
 }
 });
+
 try{
 this.parent.activeSubtests++;
 awaitthis.buildSuite;
@@ -775,19 +775,18 @@ class Suite extends Test {
 return;
 }
 
-
 if(this.parent?.hooks.beforeEach.length>0){
-awaitthis.parent[kRunHook]('beforeEach',hookArgs);
+awaitthis.parent.runHook('beforeEach',hookArgs);
 }
 
-awaitthis[kRunHook]('before',hookArgs);
+awaitthis.runHook('before',hookArgs);
 
 conststopPromise=stopTest(this.timeout,this.signal);
 constsubtests=this.skipped||this.error ? [] : this.subtests;
 constpromise=SafePromiseAll(subtests,(subtests)=>subtests.start());
 
 awaitSafePromiseRace([promise,stopPromise]);
-awaitthis[kRunHook]('after',hookArgs);
+awaitthis.runHook('after',hookArgs);
 awaitafterEach();
 
 this.pass();