[MOB-38657] Functional + BPT: Assert Dialog and Answer Dialog actions do not work

[devership16]

• Jira: https://perforce.atlassian.net/browse/MOB-38657
• Updated dialogs_replace() in bzt/resources/selenium_extras.py to stop calling the original native dialog. The override now just captures the message in a JS array, which is all it ever needed to do. The native pop-up was never useful in automated execution.
• The fix is pure JS, works across all browsers, and only affects tests using dialog actions. Verified on DEV BZM — tests pass.
• Tested via jira: https://blazect-jenkins.blazemeter.com/job/taurus-branch-builder/550/
• Regression Tests on DEV BZM
  • Scriptless: https://blazect-jenkins.blazemeter.com/job/guiTests_Scriptless/3057/
  • API GRID Test: https://blazect-jenkins.blazemeter.com/job/API_TESTS_GRID/2526/

[Copilot]

Pull request overview

Updates the Selenium JS dialog interception helpers used by Taurus-generated Selenium tests to make alert()/confirm() non-blocking and better support “assert dialog” / “answer dialog” actions.

Changes:

• Simplified the overridden window.alert() implementation to always capture messages without delegating to the native alert.
• Changed overridden window.confirm() to default to true when no explicit next-answer is set.

Comments suppressed due to low confidence (1)

bzt/resources/selenium_extras.py:246

• dialogs_replace() initializes window.__webdriverNextPrompts (plural), but the prompt override reads/writes window.__webdriverNextPrompt (singular). As a result, on the first window.prompt() call __webdriverNextPrompt may be undefined, so the overridden prompt returns undefined instead of a string or null, and dialogs_answer_on_next_prompt() may behave inconsistently. Align the variable name (and consider initializing __webdriverNextPrompt to a sensible default).

          window.__webdriverPrompts = [];
          window.__webdriverNextPrompts = true;
          window.prompt = function(msg, def) {
            window.__webdriverPrompts.push(msg || def);
            var res = window.__webdriverNextPrompt;
            window.__webdriverNextPrompt = true;
            return res;

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.90%. Comparing base (634e77c) to head (7ad8ccf).

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1984   +/-   ##
=======================================
  Coverage   89.90%   89.90%           
=======================================
  Files          71       71           
  Lines       20232    20232           
=======================================
  Hits        18187    18187           
  Misses       2045     2045           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.