What the developer wrote
Authored by Nicolas Dorier
Fix flaky email test
What changed, and why it matters
This commit adds a single line to a test file to wait for a web page element to appear before interacting with it. It is a test reliability fix only and does not change any production code, user-facing behavior, or security boundary.
No security action required. Treat as routine test maintenance.
Security signals we found
No strong security signals were identified.
Evidence from the diff
In BTCPayServer.Tests/PMO/EmailRulePMO.cs, the Fill helper now awaits s.Page.Locator(“.note-editable”).WaitForAsync() before proceeding to select options and fill form fields. This is a synchronization fix for a Playwright UI test, addressing a flaky test caused by the rich-text editor not being ready. No production code is modified.
Changed components
BTCPayServer.Tests/PMO/EmailRulePMO.csInspect captured patch +1 / −0
### BTCPayServer.Tests/PMO/EmailRulePMO.cs
@@ -32,6 +32,7 @@ public class Form
public async Task Fill(Form form)
{
+ await s.Page.Locator(".note-editable").WaitForAsync();
if (form.Trigger is not null)
await s.Page.SelectOptionAsync("#Trigger", form.Trigger);
if (form.Condition is not null)Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.