lib: added SuiteContext class

debadree25 · targos · commit 99410efd192f · 2022-12-12T16:17:47.000Z
added SuiteContext class to replace object literal Fixes: #45641 Refs: #45641 (comment) PR-URL: #45687 Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
diff --git a/lib/internal/test_runner/test.js b/lib/internal/test_runner/test.js
@@ -146,6 +146,22 @@ class TestContext {
 }
 }
 
+class SuiteContext {
+ #suite;
+
+ constructor(suite) {
+ this.#suite = suite;
+ }
+
+ get signal() {
+ return this.#suite.signal;
+ }
+
+ get name() {
+ return this.#suite.name;
+ }
+}
+
 class Test extends AsyncResource {
 #abortController;
 #outerSignal;
@@ -737,7 +753,8 @@ class Suite extends Test {
 }
 
 getRunArgs() {
- return { ctx: { signal: this.signal, name: this.name }, args: [] };
+ const ctx = new SuiteContext(this);
+ return { ctx, args: [ctx] };
 }
 
 async run() {

-Original file line number
+Diff line change
+}
+}
 +classSuiteContext{
 + #suite;
++
 +constructor(suite){
 +this.#suite =suite;
 +}
++
 +getsignal(){
 +returnthis.#suite.signal;
 +}
++
 +getname(){
 +returnthis.#suite.name;
 +}
 +}
++
 classTestextendsAsyncResource{
  #abortController;
  #outerSignal;
+}
 getRunArgs(){
 -return{ctx: {signal: this.signal,name: this.name},args: []};
 +constctx=newSuiteContext(this);
 +return{ ctx,args: [ctx]};
+}
 asyncrun(){