7c88e4d013
This change blocks off 2 common scenarios where a race condition is upgraded to a hang, when the library could internally do better. * Since we don't know whether the receiver of a `reply_to` is expecting a raw or pickled message, and since in the case of a raw reply, there is no way to signal "dead" to the receiver, override the reply_to field to explicitly mark a message as dead using a special handle. This replaces the serialized _DEAD sentinel value with a slightly neater interface, in the form of the reserved IS_DEAD handle, and enables an important subsequent change: when a context cannot route a message, it can send a generic 'dead' reply back towards the message source, ensuring any sleeping thread is woken with ChannelError. The use of this field could potentially be extended later on if additional flags are needed, but for now this seems to suffice. * Teach Router._invoke() to reply with a dead message when it receives a message for an invalid local handle. * Teach Router._async_route() to reply with a dead message when it receives an unroutable message. |
7 years ago | |
|---|---|---|
| .. | ||
| ansible | 7 years ago | |
| bench | ||
| data | 7 years ago | |
| soak | ||
| README.md | 7 years ago | |
| __init__.py | ||
| build_docker_images.py | 7 years ago | |
| call_error_test.py | ||
| call_function_test.py | 7 years ago | |
| channel_test.py | ||
| fakessh_test.py | ||
| first_stage_test.py | ||
| fork_test.py | ||
| id_allocation_test.py | ||
| importer_test.py | ||
| io_op_test.py | ||
| latch_test.py | ||
| local_test.py | ||
| master_test.py | ||
| minimize_source_test.py | ||
| module_finder_test.py | ||
| nested_test.py | ||
| parent_test.py | 7 years ago | |
| receiver_test.py | ||
| responder_test.py | ||
| router_test.py | 7 years ago | |
| select_test.py | ||
| show_docker_hostname.py | ||
| ssh_test.py | ||
| testlib.py | ||
| utils_test.py | ||
README.md
Warning
This directory is full of disorganized crap, including random hacks I checked in that I'd like to turn into tests. The effort to write tests only really started in September 2017. Pull requests in this area are very welcome!
Running The Tests
Your computer should have an Internet connection, and the docker command
line tool should be able to connect to a working Docker daemon (localhost or
elsewhere for OS X etc.) that can run new images.
The IP address of the Docker daemon must allow exposing ports from running containers, e.g. it should not be firewalled or port forwarded.
If in doubt, just install Docker on a Linux box in the default configuration and run the tests there.
Steps To Prepare Development Environment
- Get the code
git clone https://github.com/dw/mitogen.git - Go into the working directory
cd mitogen - Establish the docker image
./tests/build_docker_image.py - Build the virtual environment
virtualenv ../venv - Enable the virtual environment we just built
source ../venv/bin/activate - Install Mitogen in pip editable mode
pip install -e . - Run
test
Selecting a target distribution
Docker target images exist for testing against CentOS and Debian, with the
default being Debian. To select CentOS, specify MITOGEN_TEST_DISTRO=centos in
the environment.
User Accounts
A set of standard user account names are used by in the Docker container and
also by Ansible's osx_setup.yml.
root
In the Docker image only, the password is "rootpassword".
mitogen__has_sudo
The login password is "has_sudo_password" and the account is capable of
sudoing to root, by supplying the account password to sudo.
mitogen__has_sudo_pubkey
The login password is "has_sudo_pubkey_password". Additionally
tests/data/docker/mitogen__has_sudo_pubkey.key SSH key may be used to log
in. It can sudo the same as mitogen__has_sudo.
mitogen__has_sudo_nopw
The login password is "has_sudo_nopw_password". It can sudo to root without
supplying a password. It has explicit sudoers rules forcing it to require a
password for other accounts.
mitogen__pw_required
The login password is "pw_required_password". When "sudo -u" is used to
target this account, its password must be entered rather than the login
user's password.
mitogen__require_tty
The login password is "require_tty_password". When "sudo -u" is used to
target this account, the parent session requires a TTY.
mitogen__require_tty_pw_required
The login password is "require_tty_pw_required_password". When "sudo -u" is
used to target this account, the parent session requires a TTY and the
account password must be entered.
mitogen__user1 .. mitogen__user21
These accounts do not have passwords set. They exist to test the Ansible
interpreter recycling logic.
mitogen__webapp
A plain old account with no sudo access, used as the target for fakessh
tddests.
Ansible Integration Test Environment
The integration tests expect to be run against a either one of the Docker images, or a similar target with the same set of UNIX accounts and sudo rules.
The login account should be able to sudo to root witout a password.