ExternalVendorCode/tahoe-lafs
1
0
Fork 0
mirror of https://github.com/tahoe-lafs/tahoe-lafs.git synced 2025-01-01 10:46:42 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Get rid of the sys.exit monkey-patch

Browse Source 
It's just an exception.  Let it get logged and then check after.
This commit is contained in:
Jean-Paul Calderone 2020-12-16 21:20:16 -05:00
parent a04a915628
commit faf8da82dd
1 changed files with 3 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/allmydata/test/cli/test_cli.py
View File

@ -516,12 +516,6 @@ class CLI(CLITestMixin, unittest.TestCase):
raise exc
stderr = StringIO()
ns = Namespace()
ns.sys_exit_called = False
def call_sys_exit(exitcode):
ns.sys_exit_called = True
self.failUnlessEqual(exitcode, 1)
def fake_react(f):
reactor = Mock()
@ -530,8 +524,7 @@ class CLI(CLITestMixin, unittest.TestCase):
# it's safe to drop it on the floor.
f(reactor)
patcher = MonkeyPatcher((sys, 'exit', call_sys_exit),
(task, 'react', fake_react),
patcher = MonkeyPatcher((task, 'react', fake_react),
)
patcher.runWithPatches(
lambda: runner.run(
@ -541,8 +534,9 @@ class CLI(CLITestMixin, unittest.TestCase):
),
)
self.failUnless(ns.sys_exit_called)
self.failUnlessIn(str(exc), stderr.getvalue())
[exit_exc] = self.flushLoggedErrors(SystemExit)
self.assertEqual(1, exit_exc.value.code)
class Help(unittest.TestCase):