Follow id conventions in tests

Author: cpsievert

shiny/api-examples/MarkdownStream/app-core.py

@@ -7,7 +7,7 @@
 app_ui = ui.page_fluid(
     ui.card(
         ui.card_header("Shiny's README.md"),
-        ui.output_markdown_stream("shiny-readme"),
+        ui.output_markdown_stream("shiny_readme"),
         height="400px",
         class_="mt-3",
         full_screen=True,
@@ -28,7 +28,7 @@ async def chunk_generator():
             await asyncio.sleep(0.02)
             yield chunk + " "
 
-    md = ui.MarkdownStream("shiny-readme")
+    md = ui.MarkdownStream("shiny_readme")
 
     @reactive.effect
     async def _():

shiny/api-examples/MarkdownStream/app-express.py

@@ -8,7 +8,7 @@
 ui.page_opts(full_width=True)
 
 # Initialize a markdown stream object
-md = ui.MarkdownStream("shiny-readme")
+md = ui.MarkdownStream("shiny_readme")
 
 # Display the stream UI in a card
 with ui.card(height="400px", class_="mt-3", full_screen=True):

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")