Skip to content

Commit

Permalink
Fix failing unit test
Browse files Browse the repository at this point in the history
- Also added crash logs to gitignore
  • Loading branch information
mofojed committed Feb 29, 2024
1 parent 3cef448 commit 4c0bc7b
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
4 changes: 4 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -24,3 +24,7 @@ docker-compose.override.yml

# Ignore temporary files created during a release
releases/

# virtual machine crash logs, see http://www.java.com/en/download/help/error_hotspot.xml
hs_err_pid*
replay_pid*
4 changes: 2 additions & 2 deletions plugins/ui/test/deephaven/ui/test_hooks.py
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,7 @@ def listener(update: TableUpdate, is_replay: bool) -> None:
event.set()

def _test_table_listener(replayed_table_val=table, listener_val=listener):
use_table_listener(replayed_table_val, listener_val, set())
use_table_listener(replayed_table_val, listener_val, [])

render_hook(_test_table_listener)

Expand All @@ -171,7 +171,7 @@ def listener(update: TableUpdate, is_replay: bool) -> None:
event.set()

def _test_table_listener(replay_table=table, listener_val=listener):
use_table_listener(replay_table, listener_val, do_replay=True)
use_table_listener(replay_table, listener_val, [], do_replay=True)

render_hook(_test_table_listener)

Expand Down

0 comments on commit 4c0bc7b

Please sign in to comment.