Skip to content

Commit ae85f7a

Browse files
committed
Replace test_signal_management.py __noop handler with standard default_int_handler to match how signals are now handled in Launch
Signed-off-by: Cian Donovan <[email protected]>
1 parent 43115f7 commit ae85f7a

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

launch/test/launch/utilities/test_signal_management.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -25,19 +25,19 @@
2525

2626

2727
def cap_signals(*signals):
28-
def _noop(*args):
29-
pass
30-
3128
def _decorator(func):
3229
@functools.wraps(func)
3330
def _wrapper(*args, **kwargs):
3431
handlers = {}
3532
try:
3633
for s in signals:
37-
handlers[s] = signal.signal(s, _noop)
34+
handlers[s] = signal.signal(s, signal.default_int_handler)
3835
return func(*args, **kwargs)
36+
except KeyboardInterrupt:
37+
pass
3938
finally:
40-
assert all(signal.signal(s, h) is _noop for s, h in handlers.items())
39+
assert all(signal.signal(s, h) is signal.default_int_handler
40+
for s, h in handlers.items())
4141
return _wrapper
4242

4343
return _decorator

0 commit comments

Comments
 (0)