test: update renderSidebar regex to match 3-parameter signature
The test regexes matched `renderSidebar(\w+,\s*\w+)` (exactly 2 params)
but the fix in 821c595 added a third `currentRemoteId` parameter.
Updated to use `\(.*?\)` which matches any parameter count.
Generated with Amplifier
Co-Authored-By: Amplifier <240397093+microsoft-amplifier@users.noreply.github.com>
This commit is contained in:
@@ -2244,7 +2244,7 @@ def test_render_grid_reads_sort_order_from_server_settings() -> None:
|
|||||||
def test_render_sidebar_filters_hidden_sessions() -> None:
|
def test_render_sidebar_filters_hidden_sessions() -> None:
|
||||||
"""renderSidebar() must filter out hidden sessions via getVisibleSessions()."""
|
"""renderSidebar() must filter out hidden sessions via getVisibleSessions()."""
|
||||||
match = re.search(
|
match = re.search(
|
||||||
r"function renderSidebar\s*\(\w+,\s*\w+\)\s*\{(.*?)(?=\nconst SIDEBAR_KEY|function |\n// ─)",
|
r"function renderSidebar\s*\(.*?\)\s*\{(.*?)(?=\nconst SIDEBAR_KEY|function |\n// ─)",
|
||||||
_JS,
|
_JS,
|
||||||
re.DOTALL,
|
re.DOTALL,
|
||||||
)
|
)
|
||||||
@@ -2258,7 +2258,7 @@ def test_render_sidebar_filters_hidden_sessions() -> None:
|
|||||||
def test_render_sidebar_uses_visible_array() -> None:
|
def test_render_sidebar_uses_visible_array() -> None:
|
||||||
"""renderSidebar() must use visible array for rendering, not original sessions."""
|
"""renderSidebar() must use visible array for rendering, not original sessions."""
|
||||||
match = re.search(
|
match = re.search(
|
||||||
r"function renderSidebar\s*\(\w+,\s*\w+\)\s*\{(.*?)(?=\nconst SIDEBAR_KEY|function |\n// ─)",
|
r"function renderSidebar\s*\(.*?\)\s*\{(.*?)(?=\nconst SIDEBAR_KEY|function |\n// ─)",
|
||||||
_JS,
|
_JS,
|
||||||
re.DOTALL,
|
re.DOTALL,
|
||||||
)
|
)
|
||||||
|
|||||||
Reference in New Issue
Block a user