From f98279fc95fb1900deec10fe978da467ac2c1ae0 Mon Sep 17 00:00:00 2001 From: David Wilson Date: Fri, 4 May 2018 17:38:49 +0100 Subject: [PATCH] tests: fix LRU test after splitting up Connection class. The module the connection class is now loaded as is "ansible.plugins.connection.mitogen_ssh", etc., which breaks the test. Instead, check if the connection is an instance of the base Connection class. --- tests/ansible/lib/action/mitogen_shutdown_all.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tests/ansible/lib/action/mitogen_shutdown_all.py b/tests/ansible/lib/action/mitogen_shutdown_all.py index 65be8b20..9af21e11 100644 --- a/tests/ansible/lib/action/mitogen_shutdown_all.py +++ b/tests/ansible/lib/action/mitogen_shutdown_all.py @@ -6,6 +6,7 @@ required for reliable LRU tests. import traceback import sys +import ansible_mitogen.connection import ansible_mitogen.services import mitogen.service @@ -15,9 +16,10 @@ from ansible.plugins.action import ActionBase class ActionModule(ActionBase): def run(self, tmp=None, task_vars=None): - if not type(self._connection).__module__.startswith('ansible_mitogen'): + if not isinstance(self._connection, + ansible_mitogen.connection.Connection): return { - 'changed': False + 'skipped': True, } self._connection._connect()