Fix for non-str ident (#1268)

ansible · Jul 12, 2023 · 8301351 · 8301351
1 parent aef73cf
commit 8301351
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/src/ansible_runner/streaming.py b/src/ansible_runner/streaming.py
@@ -267,7 +267,7 @@ def __init__(self, _input=None, status_handler=None, event_handler=None,
         else:
             project_artifacts = os.path.abspath(os.path.join(self.private_data_dir, 'artifacts'))
             if ident := kwargs.get('ident'):
-                self.artifact_dir = os.path.join(project_artifacts, ident)
+                self.artifact_dir = os.path.join(project_artifacts, str(ident))
             else:
                 self.artifact_dir = project_artifacts
 

diff --git a/test/unit/test_streaming.py b/test/unit/test_streaming.py
@@ -0,0 +1,16 @@
+import os
+
+from ansible_runner.streaming import Processor
+
+
+class TestProcessor:
+
+    def test_artifact_dir_with_int_ident(self, tmp_path):
+        kwargs = {
+            'private_data_dir': str(tmp_path),
+            'ident': 123,
+        }
+        p = Processor(**kwargs)
+        assert p.artifact_dir == os.path.join(kwargs['private_data_dir'],
+                                              'artifacts',
+                                              str(kwargs['ident']))