Skip to content

Commit 0d321fd

Browse files
committed
Clarify test error rmessage.
And precompute the delta-time as pytest's: > assert (1728139272.6514266 - 1728139262.6307526) < 5 Is not super useful to know how long the timeout was. In particular it allos us to see in ipykernels' main branch that the delta is >=10s, so that the interrupt call is not properly handled.
1 parent f9fc2f6 commit 0d321fd

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

spyder_kernels/console/tests/test_console_kernel.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1314,13 +1314,17 @@ def test_interrupt():
13141314
kernel_comm.remote_call().raise_interrupt_signal()
13151315
# Wait for shell message
13161316
while True:
1317-
assert time.time() - t0 < 5
1317+
delta = time.time() - t0
1318+
assert delta < 5
13181319
msg = client.get_shell_msg(timeout=TIMEOUT)
13191320
if msg["parent_header"].get("msg_id") != msg_id:
13201321
# not from my request
13211322
continue
13221323
break
1323-
assert time.time() - t0 < 5
1324+
delta = time.time() - t0
1325+
assert (
1326+
delta < 5
1327+
), "10 second long call should have been interupted, interrupt signal was likely misshandled"
13241328

13251329

13261330
def test_enter_debug_after_interruption():

0 commit comments

Comments
 (0)