rusackas commented on code in PR #44102:
URL: https://github.com/apache/superset/pull/44102#discussion_r4288638742
##########
tests/unit_tests/utils/test_screenshot_utils.py:
##########
@@ -1867,3 +1867,109 @@ def
test_expand_scrollable_content_js_unrolls_ag_grid_and_css_scroll() -> None:
# bounded by a report's remaining deadline (see webdriver_test.py).
assert "async (maxWaitMs) =>" in EXPAND_SCROLLABLE_CONTENT_JS
assert "Date.now() + maxWaitMs" in EXPAND_SCROLLABLE_CONTENT_JS
+
+ # The generic-descendant reset repeats to a fixed point (bounded), not a
+ # single pass: an ancestor whose own height was computed as the sum of
+ # its still-clipped children (e.g. plugin-chart-table's `role="table"`
+ # sticky wrapper in useSticky.tsx) has scrollHeight == clientHeight
+ # *before* its child is expanded, so a single querySelectorAll pass
+ # skips it -- see
test_expand_scrollable_content_resolves_nested_ancestor_clip
+ # below for the real-browser reproduction.
+ assert "changedInPass" in EXPAND_SCROLLABLE_CONTENT_JS
+ assert "passes < 5" in EXPAND_SCROLLABLE_CONTENT_JS
Review Comment:
Good find, confirmed both of those survive the old asserts. Replaced the two
substring checks with one exact match of the whole control-flow block
(c4537218e5) and reran your two mutations against it, both fail now. The
real-browser Playwright test still carries the actual behavioral proof, this
just closes the gap on what's CI-enforced.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
