|
|
|
import mock
|
|
|
|
import textwrap
|
|
|
|
import subprocess
|
|
|
|
import sys
|
|
|
|
import unittest
|
|
|
|
|
|
|
|
import mitogen.master
|
|
|
|
import testlib
|
|
|
|
|
|
|
|
import plain_old_module
|
|
|
|
import simple_pkg.a
|
|
|
|
|
|
|
|
|
|
|
|
class NeutralizeMainTest(testlib.RouterMixin, testlib.TestCase):
|
|
|
|
klass = mitogen.master.ModuleResponder
|
|
|
|
|
|
|
|
def call(self, *args, **kwargs):
|
|
|
|
router = mock.Mock()
|
|
|
|
return self.klass(router).neutralize_main(*args, **kwargs)
|
|
|
|
|
|
|
|
def test_missing_exec_guard(self):
|
|
|
|
path = testlib.data_path('main_with_no_exec_guard.py')
|
|
|
|
args = [sys.executable, path]
|
|
|
|
proc = subprocess.Popen(args, stderr=subprocess.PIPE)
|
|
|
|
_, stderr = proc.communicate()
|
|
|
|
self.assertEqual(1, proc.returncode)
|
|
|
|
expect = self.klass.main_guard_msg % (path,)
|
|
|
|
self.assertIn(expect, stderr.decode())
|
|
|
|
|
|
|
|
HAS_MITOGEN_MAIN = mitogen.core.b(
|
|
|
|
textwrap.dedent("""
|
|
|
|
herp derp
|
|
|
|
|
|
|
|
def myprog():
|
|
|
|
pass
|
|
|
|
|
|
|
|
@mitogen.main(maybe_some_option=True)
|
|
|
|
def main(router):
|
|
|
|
pass
|
|
|
|
""")
|
|
|
|
)
|
|
|
|
|
|
|
|
def test_mitogen_main(self):
|
|
|
|
untouched = self.call("derp.py", self.HAS_MITOGEN_MAIN)
|
|
|
|
self.assertEqual(untouched, self.HAS_MITOGEN_MAIN)
|
|
|
|
|
|
|
|
HAS_EXEC_GUARD = mitogen.core.b(
|
|
|
|
textwrap.dedent("""
|
|
|
|
herp derp
|
|
|
|
|
|
|
|
def myprog():
|
|
|
|
pass
|
|
|
|
|
|
|
|
def main():
|
|
|
|
pass
|
|
|
|
|
|
|
|
if __name__ == '__main__':
|
|
|
|
main()
|
|
|
|
""")
|
|
|
|
)
|
|
|
|
|
|
|
|
def test_exec_guard(self):
|
|
|
|
touched = self.call("derp.py", self.HAS_EXEC_GUARD)
|
|
|
|
bits = touched.decode().split()
|
|
|
|
self.assertEqual(bits[-3:], ['def', 'main():', 'pass'])
|
|
|
|
|
|
|
|
|
|
|
|
class GoodModulesTest(testlib.RouterMixin, testlib.TestCase):
|
|
|
|
def test_plain_old_module(self):
|
|
|
|
# The simplest case: a top-level module with no interesting imports or
|
|
|
|
# package machinery damage.
|
|
|
|
context = self.router.local()
|
|
|
|
|
|
|
|
self.assertEqual(256, context.call(plain_old_module.pow, 2, 8))
|
|
|
|
os_fork = int(sys.version_info < (2, 6)) # mitogen.os_fork
|
|
|
|
self.assertEqual(1+os_fork, self.router.responder.get_module_count)
|
|
|
|
self.assertEqual(1+os_fork, self.router.responder.good_load_module_count)
|
|
|
|
self.assertLess(300, self.router.responder.good_load_module_size)
|
|
|
|
|
|
|
|
def test_simple_pkg(self):
|
|
|
|
# Ensure success of a simple package containing two submodules, one of
|
|
|
|
# which imports the other.
|
|
|
|
context = self.router.local()
|
|
|
|
self.assertEqual(3,
|
|
|
|
context.call(simple_pkg.a.subtract_one_add_two, 2))
|
|
|
|
os_fork = int(sys.version_info < (2, 6)) # mitogen.os_fork
|
|
|
|
self.assertEqual(2+os_fork, self.router.responder.get_module_count)
|
|
|
|
self.assertEqual(3+os_fork, self.router.responder.good_load_module_count)
|
|
|
|
self.assertEqual(0, self.router.responder.bad_load_module_count)
|
|
|
|
self.assertLess(450, self.router.responder.good_load_module_size)
|
|
|
|
|
|
|
|
def test_self_contained_program(self):
|
|
|
|
# Ensure a program composed of a single script can be imported
|
|
|
|
# successfully.
|
|
|
|
args = [sys.executable, testlib.data_path('self_contained_program.py')]
|
|
|
|
proc = subprocess.Popen(args, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
|
|
|
|
b_stdout, _ = proc.communicate()
|
|
|
|
self.assertEqual(proc.returncode, 0)
|
|
|
|
self.assertEqual(b_stdout.decode(), "['__main__', 50]\n")
|
|
|
|
|
|
|
|
|
|
|
|
class BrokenModulesTest(testlib.TestCase):
|
|
|
|
def test_obviously_missing(self):
|
|
|
|
# Ensure we don't crash in the case of a module legitimately being
|
|
|
|
# unavailable. Should never happen in the real world.
|
|
|
|
|
|
|
|
stream = mock.Mock()
|
Refactor Stream, introduce quasi-asynchronous connect, much more
Split Stream into many, many classes
* mitogen.parent.Connection: Handles connection setup logic only.
* Maintain references to stdout and stderr streams.
* Manages TimerList timer to cancel connection attempt after
deadline
* Blocking setup code replaced by async equivalents running on the
broker
* mitogen.parent.Options: Tracks connection-specific options. This
keeps the connection class small, but more importantly, it is
generic to the future desire to build and execute command lines
without starting a full connection.
* mitogen.core.Protocol: Handles program behaviour relating to events
on a stream. Protocol performs no IO of its own, instead deferring
it to Stream and Side. This makes testing much easier, and means
libssh can reimplement Stream and Side to reuse MitogenProtocol
* mitogen.core.MitogenProtocol: Guts of the old Mitogen stream
implementtion
* mitogen.core.BufferedWriter: Guts of the old Mitogen buffered
transmit implementation, made generic
* mitogen.core.DelineatedProtocol: Guts of the old IoLogger, knows how
to split up input and pass it on to a
on_line_received()/on_partial_line_received() callback.
* mitogen.parent.BootstrapProtocol: Asynchronous equivalent of the old
blocking connect code. Waits for various prompts (MITO001 etc) and
writes the bootstrap using a BufferedWriter. On success, switches
the stream to MitogenProtocol.
* mitogen.core.Message: move encoding parts of MitogenProtocol out to
Message (where it belongs) and write a bunch of new tests for
pickling.
* The bizarre Stream.construct() is gone now, Option.__init__ is its
own constructor. Should fix many LGTM errors.
* Update all connection methods: Every connection method is updated to
use async logic, defining protocols as required to handle interactive
prompts like in SSH or su. Add new real integration tests for at least
doas and su.
* Eliminate manual fd management: File descriptors are trapped in file
objects at their point of origin, and Side is updated to use file
objects rather than raw descriptors. This eliminates a whole class of
bugs where unrelated FDs could be closed by the wrong component. Now
an FD's open/closed status is fused to it everywhere in the library.
* Halve file descriptor usage: now FD open/close state is tracked by
its file object, we don't need to duplicate FDs everywhere so that
receive/transmit side can be closed independently. Instead both sides
back on to the same file object. Closes #26, Closes #470.
* Remove most uses of dup/dup2: Closes #256. File descriptors are
trapped in a common file object and shared among classes. The
remaining few uses for dup/dup2 are as close to minimal as possible.
* Introduce mitogen.parent.Process: uniform interface for subprocesses
created either via mitogen.fork or the subprocess module. Remove all
the crap where we steal a pid from subprocess guts. Now we use
subprocess to manage its processes as it should be. Closes #169 by
using the new Timers facility to poll for a slow-to-exit subprocess.
* Fix su password race: Closes #363. DelineatedProtocol naturally
retries partially received lines, preventing the cause of the original
race.
* Delete old blocking IO utility functions
iter_read()/write_all()/discard_until().
Closes #26
Closes #147
Closes #169
Closes #256
Closes #363
Closes #419
Closes #470
6 years ago
|
|
|
stream.protocol.sent_modules = set()
|
|
|
|
router = mock.Mock()
|
|
|
|
router.stream_by_id = lambda n: stream
|
|
|
|
|
|
|
|
msg = mitogen.core.Message(
|
|
|
|
data=mitogen.core.b('non_existent_module'),
|
|
|
|
reply_to=50,
|
|
|
|
)
|
|
|
|
msg.router = router
|
|
|
|
|
|
|
|
responder = mitogen.master.ModuleResponder(router)
|
|
|
|
responder._on_get_module(msg)
|
|
|
|
self.assertEqual(1, len(router._async_route.mock_calls))
|
|
|
|
|
|
|
|
self.assertEqual(1, responder.get_module_count)
|
|
|
|
self.assertEqual(0, responder.good_load_module_count)
|
|
|
|
self.assertEqual(0, responder.good_load_module_size)
|
|
|
|
self.assertEqual(1, responder.bad_load_module_count)
|
|
|
|
|
|
|
|
call = router._async_route.mock_calls[0]
|
|
|
|
msg, = call[1]
|
|
|
|
self.assertEqual(mitogen.core.LOAD_MODULE, msg.handle)
|
|
|
|
self.assertEqual(('non_existent_module', None, None, None, ()),
|
|
|
|
msg.unpickle())
|
|
|
|
|
|
|
|
@unittest.skipIf(
|
|
|
|
condition=sys.version_info < (2, 6),
|
|
|
|
reason='Ancient Python lacked "from . import foo"',
|
|
|
|
)
|
|
|
|
def test_ansible_six_messed_up_path(self):
|
|
|
|
# The copy of six.py shipped with Ansible appears in a package whose
|
|
|
|
# __path__ subsequently ends up empty, which prevents pkgutil from
|
|
|
|
# finding its submodules. After ansible.compat.six is initialized in
|
|
|
|
# the parent, attempts to execute six/__init__.py on the slave will
|
|
|
|
# cause an attempt to request ansible.compat.six._six from the master.
|
|
|
|
import six_brokenpkg
|
|
|
|
|
|
|
|
stream = mock.Mock()
|
Refactor Stream, introduce quasi-asynchronous connect, much more
Split Stream into many, many classes
* mitogen.parent.Connection: Handles connection setup logic only.
* Maintain references to stdout and stderr streams.
* Manages TimerList timer to cancel connection attempt after
deadline
* Blocking setup code replaced by async equivalents running on the
broker
* mitogen.parent.Options: Tracks connection-specific options. This
keeps the connection class small, but more importantly, it is
generic to the future desire to build and execute command lines
without starting a full connection.
* mitogen.core.Protocol: Handles program behaviour relating to events
on a stream. Protocol performs no IO of its own, instead deferring
it to Stream and Side. This makes testing much easier, and means
libssh can reimplement Stream and Side to reuse MitogenProtocol
* mitogen.core.MitogenProtocol: Guts of the old Mitogen stream
implementtion
* mitogen.core.BufferedWriter: Guts of the old Mitogen buffered
transmit implementation, made generic
* mitogen.core.DelineatedProtocol: Guts of the old IoLogger, knows how
to split up input and pass it on to a
on_line_received()/on_partial_line_received() callback.
* mitogen.parent.BootstrapProtocol: Asynchronous equivalent of the old
blocking connect code. Waits for various prompts (MITO001 etc) and
writes the bootstrap using a BufferedWriter. On success, switches
the stream to MitogenProtocol.
* mitogen.core.Message: move encoding parts of MitogenProtocol out to
Message (where it belongs) and write a bunch of new tests for
pickling.
* The bizarre Stream.construct() is gone now, Option.__init__ is its
own constructor. Should fix many LGTM errors.
* Update all connection methods: Every connection method is updated to
use async logic, defining protocols as required to handle interactive
prompts like in SSH or su. Add new real integration tests for at least
doas and su.
* Eliminate manual fd management: File descriptors are trapped in file
objects at their point of origin, and Side is updated to use file
objects rather than raw descriptors. This eliminates a whole class of
bugs where unrelated FDs could be closed by the wrong component. Now
an FD's open/closed status is fused to it everywhere in the library.
* Halve file descriptor usage: now FD open/close state is tracked by
its file object, we don't need to duplicate FDs everywhere so that
receive/transmit side can be closed independently. Instead both sides
back on to the same file object. Closes #26, Closes #470.
* Remove most uses of dup/dup2: Closes #256. File descriptors are
trapped in a common file object and shared among classes. The
remaining few uses for dup/dup2 are as close to minimal as possible.
* Introduce mitogen.parent.Process: uniform interface for subprocesses
created either via mitogen.fork or the subprocess module. Remove all
the crap where we steal a pid from subprocess guts. Now we use
subprocess to manage its processes as it should be. Closes #169 by
using the new Timers facility to poll for a slow-to-exit subprocess.
* Fix su password race: Closes #363. DelineatedProtocol naturally
retries partially received lines, preventing the cause of the original
race.
* Delete old blocking IO utility functions
iter_read()/write_all()/discard_until().
Closes #26
Closes #147
Closes #169
Closes #256
Closes #363
Closes #419
Closes #470
6 years ago
|
|
|
stream.protocol.sent_modules = set()
|
|
|
|
router = mock.Mock()
|
|
|
|
router.stream_by_id = lambda n: stream
|
|
|
|
|
|
|
|
msg = mitogen.core.Message(
|
|
|
|
data=mitogen.core.b('six_brokenpkg._six'),
|
|
|
|
reply_to=50,
|
|
|
|
)
|
|
|
|
msg.router = router
|
|
|
|
|
|
|
|
responder = mitogen.master.ModuleResponder(router)
|
|
|
|
responder._on_get_module(msg)
|
|
|
|
self.assertEqual(1, len(router._async_route.mock_calls))
|
|
|
|
|
|
|
|
self.assertEqual(1, responder.get_module_count)
|
|
|
|
self.assertEqual(1, responder.good_load_module_count)
|
|
|
|
self.assertEqual(0, responder.bad_load_module_count)
|
|
|
|
|
|
|
|
call = router._async_route.mock_calls[0]
|
|
|
|
msg, = call[1]
|
|
|
|
self.assertEqual(mitogen.core.LOAD_MODULE, msg.handle)
|
|
|
|
|
|
|
|
tup = msg.unpickle()
|
|
|
|
self.assertIsInstance(tup, tuple)
|
|
|
|
|
|
|
|
|
|
|
|
class ForwardTest(testlib.RouterMixin, testlib.TestCase):
|
|
|
|
def test_forward_to_nonexistent_context(self):
|
|
|
|
nonexistent = mitogen.core.Context(self.router, 123)
|
|
|
|
capture = testlib.LogCapturer()
|
|
|
|
capture.start()
|
|
|
|
self.broker.defer_sync(lambda:
|
|
|
|
self.router.responder.forward_modules(
|
|
|
|
nonexistent,
|
|
|
|
['mitogen.core']
|
|
|
|
)
|
|
|
|
)
|
|
|
|
s = capture.stop()
|
|
|
|
self.assertIn('dropping forward of', s)
|
|
|
|
|
|
|
|
def test_stats(self):
|
|
|
|
# Forwarding stats broken because forwarding is broken. See #469.
|
|
|
|
c1 = self.router.local()
|
|
|
|
c2 = self.router.local(via=c1)
|
|
|
|
|
|
|
|
os_fork = int(sys.version_info < (2, 6))
|
|
|
|
self.assertEqual(256, c2.call(plain_old_module.pow, 2, 8))
|
|
|
|
self.assertEqual(2+os_fork, self.router.responder.get_module_count)
|
|
|
|
self.assertEqual(2+os_fork, self.router.responder.good_load_module_count)
|
|
|
|
self.assertLess(10000, self.router.responder.good_load_module_size)
|
|
|
|
self.assertGreater(40000, self.router.responder.good_load_module_size)
|
|
|
|
|
|
|
|
|
|
|
|
class BlacklistTest(testlib.TestCase):
|
|
|
|
@unittest.skip('implement me')
|
|
|
|
def test_whitelist_no_blacklist(self):
|
|
|
|
assert 0
|
|
|
|
|
|
|
|
@unittest.skip('implement me')
|
|
|
|
def test_whitelist_has_blacklist(self):
|
|
|
|
assert 0
|
|
|
|
|
|
|
|
@unittest.skip('implement me')
|
|
|
|
def test_blacklist_no_whitelist(self):
|
|
|
|
assert 0
|
|
|
|
|
|
|
|
@unittest.skip('implement me')
|
|
|
|
def test_blacklist_has_whitelist(self):
|
|
|
|
assert 0
|