Skip to content

fix: async context leaking#1225

Open
KuznetsovRoman wants to merge 1 commit intomasterfrom
TESTPLANE-950.async_ctx
Open

fix: async context leaking#1225
KuznetsovRoman wants to merge 1 commit intomasterfrom
TESTPLANE-950.async_ctx

Conversation

@KuznetsovRoman
Copy link
Member

No description provided.

Comment on lines +35 to +37
return new Promise((resolve, reject) => {
this._queue = this._queue.finally(() => this._parser.parse({ file, browserId }).then(resolve, reject));
});
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given that fastq concurrency was 1, it is the same thing.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 25, 2026

Open in StackBlitz

npm i https://pkg.pr.new/gemini-testing/testplane@1225

commit: 4e4a148

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants