Skip to content

fix: async context leaking#1225

Merged
KuznetsovRoman merged 1 commit intomasterfrom
TESTPLANE-950.async_ctx
Mar 27, 2026
Merged

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

Conversation

@KuznetsovRoman
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown

pkg-pr-new bot commented Mar 25, 2026

Open in StackBlitz

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

commit: 21c5b68

@KuznetsovRoman KuznetsovRoman force-pushed the TESTPLANE-950.async_ctx branch from 4e4a148 to 21c5b68 Compare March 27, 2026 12:20
@KuznetsovRoman KuznetsovRoman merged commit 8bffc4d into master Mar 27, 2026
7 checks passed
@KuznetsovRoman KuznetsovRoman deleted the TESTPLANE-950.async_ctx branch March 27, 2026 12:33
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