Skip to content

Conversation

@karangattu
Copy link
Collaborator

@karangattu karangattu commented Mar 21, 2024

Changed app to use python list rather than stderr/stdout. Using a page.refresh(), we can trigger the "on session ended" functions. Using the list, we can display it in a text output which is easily verifiable with playwright helpers.

@karangattu karangattu changed the title See error test: Fix broken test on main Mar 21, 2024
@karangattu karangattu marked this pull request as ready for review March 26, 2024 19:49
@karangattu karangattu requested a review from schloerke March 26, 2024 19:49
@schloerke schloerke changed the title test: Fix broken test on main test(session ended): Update test to use variable, not capture stdout Mar 26, 2024
@schloerke schloerke changed the title test(session ended): Update test to use variable, not capture stdout test: Update test to use variable, not capture stdout Mar 26, 2024
@schloerke schloerke merged commit 5faf62b into main Mar 26, 2024
@schloerke schloerke deleted the flush-tests branch March 26, 2024 20:14
schloerke added a commit that referenced this pull request Mar 27, 2024
* main:
  feat: Create `Session.set_message_handler(name, handler)` (#1253)
  Fix input name
  Enable error console when running locally. (#1060)
  test: Update test to use variable, not capture stdout (#1236)
schloerke added a commit that referenced this pull request Mar 28, 2024
* main:
  test(CI): Add merge queue support for faster PR testing (#1265)
  fix(panel_conditional): Include `.shiny-panel-conditional` class (#1257)
  feat: Create `Session.set_message_handler(name, handler)` (#1253)
  Fix input name
  Enable error console when running locally. (#1060)
  test: Update test to use variable, not capture stdout (#1236)
  test: Fix deploys for shinyapps.io and connect (#1224)
  chore(test): Playwright test refactors and clean up (#1204)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants