[Bf-blender-cvs] [ac41215f074] master: Tests: fix Alembic export test on Windows

Sybren A. Stüvel noreply at git.blender.org
Mon Aug 17 20:08:14 CEST 2020


Commit: ac41215f074f37859a44adeef04711c24572ebc1
Author: Sybren A. Stüvel
Date:   Mon Aug 17 20:07:45 2020 +0200
Branches: master
https://developer.blender.org/rBac41215f074f37859a44adeef04711c24572ebc1

Tests: fix Alembic export test on Windows

Convert all `subprocess.run()` arguments to string, to prevent breaking
"does this argument need quoting" checks on Windows.

===================================================================

M	tests/python/alembic_export_tests.py

===================================================================

diff --git a/tests/python/alembic_export_tests.py b/tests/python/alembic_export_tests.py
index cacd2f2dd0e..b5d6bf65f3f 100644
--- a/tests/python/alembic_export_tests.py
+++ b/tests/python/alembic_export_tests.py
@@ -67,7 +67,9 @@ class AbstractAlembicTest(AbstractBlenderRunnerTest):
         """
 
         command = (self.alembic_root / 'bin' / 'abcls', *arguments)
-        proc = subprocess.run(command, stdout=subprocess.PIPE, stderr=subprocess.STDOUT,
+        # Convert Path to str; Path works fine on Linux, but not on Windows.
+        command_str = [str(arg) for arg in command]
+        proc = subprocess.run(command_str, stdout=subprocess.PIPE, stderr=subprocess.STDOUT,
                               timeout=30)
 
         coloured_output = proc.stdout



More information about the Bf-blender-cvs mailing list