issue #454: fix AttributeError and atexit.yml test.

ansible_mitogen/runner.py

@@ -357,10 +357,10 @@ class Runner(object):
 
 class AtExitWrapper(object):
     """
-    Newer Ansibles use :func:`atexit.register` to trigger tmpdir cleanup when
+    issue #397, #454: Newer Ansibles use :func:`atexit.register` to trigger
-    AnsibleModule.tmpdir is responsible for creating its own temporary
+    tmpdir cleanup when AnsibleModule.tmpdir is responsible for creating its
-    directory, however with Mitogen processes are preserved across tasks,
+    own temporary directory, however with Mitogen processes are preserved
-    meaning cleanup must happen earlier.
+    across tasks, meaning cleanup must happen earlier.
 
     Patch :func:`atexit.register`, catching :func:`shutil.rmtree` calls so they
     can be executed on task completion, rather than on process shutdown.
@@ -401,7 +401,7 @@ class AtExitWrapper(object):
             self.deferred.append((func, targs, kwargs))
             return
 
-        self.original_register(func, *targs, **kwargs)
+        self.original['register'](func, *targs, **kwargs)
 
 
 class ModuleUtilsImporter(object):

tests/ansible/integration/runner/atexit.yml

@@ -1,10 +1,13 @@
-# issue #397: newer Ansibles rely on atexit to cleanup their temporary
+# issue #397, #454: newer Ansibles rely on atexit to cleanup their temporary
-# directories. Ensure atexit handlers run during runner completion.
+# directories. Ensure atexit handlers calling shutil.rmtree() run during runner
+# completion.
 
 - name: integration/runner/atexit.yml
   hosts: test-targets
   gather_facts: false
   any_errors_fatal: false
+  vars:
+    path: /tmp/atexit-should-delete-this
   tasks:
 
     #
@@ -14,18 +17,15 @@
 
     - custom_python_run_script:
         script: |
-          import atexit
+          import atexit, shutil
-          atexit.register(lambda:
+          path = '{{path}}'
-            open('/tmp/atexit-was-triggered', 'w').write('yep'))
+          os.mkdir(path, int('777', 8))
+          atexit.register(shutil.rmtree, path)
 
-    - slurp:
+    - stat:
-        path: /tmp/atexit-was-triggered
+        path: "{{path}}"
       register: out
 
     - assert:
         that:
-        - out.content|b64decode == "yep"
+        - not out.stat.exists
-
-    - file:
-        path: /tmp/atexit-was-triggered
-        state: absent