fix: Use page.locator in Playwright reports

[kgabryje]

SUMMARY

Per Playwright's docs, page.wait_for_selector method is deprecated. This PR replaces usages of that method with page.locator suggested by the docs.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

1. Enable PLAYWRIGHT_REPORTS_AND_THUMBNAILS feature flag
2. Run some reports and verify that it works like before

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[codecov]

Codecov Report

Attention: 14 lines in your changes are missing coverage. Please review.

Comparison is base (07e5fe8) 69.19% compared to head (55910a7) 69.17%.

Files	Patch %	Lines
superset/utils/webdriver.py	6.66%	14 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master   #26224      +/-   ##
==========================================
- Coverage   69.19%   69.17%   -0.03%     
==========================================
  Files        1945     1945              
  Lines       75938    75946       +8     
  Branches     8458     8458              
==========================================
- Hits        52545    52533      -12     
- Misses      21208    21228      +20     
  Partials     2185     2185              

Flag	Coverage Δ
hive	53.67% <12.50%> (-0.02%)	⬇️
mysql	?
postgres	78.18% <12.50%> (-0.02%)	⬇️
presto	53.62% <12.50%> (-0.02%)	⬇️
python	82.83% <12.50%> (-0.06%)	⬇️
sqlite	76.83% <12.50%> (-0.02%)	⬇️
unit	55.80% <12.50%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[geido]

I assume all of these .all() methods are safe for the loop and will just return empty if none are present

[kgabryje]

Yes, page.locator returns a Locator object, not None or list of html elements. Calling .all() on Locator that points to nothing returns an empty array.