Follow id conventions in tests

Author: cpsievert

tests/playwright/shiny/components/MarkdownStream/basic/app.py

@@ -10,8 +10,8 @@
     readme_chunks = f.read().replace("\n", " \n ").split(" ")
 
 
-stream = ui.MarkdownStream("shiny-readme")
-stream_err = ui.MarkdownStream("shiny-readme-err")
+stream = ui.MarkdownStream("shiny_readme")
+stream_err = ui.MarkdownStream("shiny_readme_err")
 
 
 async def readme_generator():
@@ -46,7 +46,7 @@ async def _():
     stream_err.ui()
 
 
-basic_stream = ui.MarkdownStream("basic-stream-result")
+basic_stream = ui.MarkdownStream("basic_stream_result")
 basic_stream.ui()
 
 

tests/playwright/shiny/components/MarkdownStream/basic/test_stream_basic.py

@@ -28,7 +28,7 @@ def is_element_scrolled_to_bottom(page: Page, selector: str) -> bool:
 def test_validate_stream_basic(page: Page, local_app: ShinyAppProc) -> None:
     page.goto(local_app.url)
 
-    stream = page.locator("#shiny-readme")
+    stream = page.locator("#shiny_readme")
     expect(stream).to_be_visible(timeout=30 * 1000)
     expect(stream).to_contain_text("pip install shiny")
 
@@ -37,7 +37,7 @@ def test_validate_stream_basic(page: Page, local_app: ShinyAppProc) -> None:
     is_scrolled = is_element_scrolled_to_bottom(page, ".card-body")
     assert is_scrolled, "The card body container should be scrolled to the bottom"
 
-    stream2 = page.locator("#shiny-readme-err")
+    stream2 = page.locator("#shiny_readme_err")
     expect(stream2).to_be_visible(timeout=30 * 1000)
     expect(stream2).to_contain_text("Shiny")