Sybren A. Stüvel
be5c9d45bd
CentOS on the buildbot still runs Python 3.6, which is also used for the
unit tests. This means that the tests can't use language features that
are available to Blender itself. And testing with a different version of
Python than will be used by the actual code seems like a bad idea to me.
This commit adds `TEST_PYTHON_EXECUTABLE` as advanced CMake option. This
will allow us to set a specific Python executable when we need it. When
not set, a platform-specific default will be used:
- On Windows, the `python….exe` from the installation directory. This is
just like before this patch, except that this patch adds the
overridability.
- On macOS/Linux, the `${PYTHON_EXECUTABLE}` as found by CMake.
Every platform should now have a value (configured by the user or
detected by CMake) for `TEST_PYTHON_EXE`, so there is no need to allow
running without. This also removes the need to have some Python files
marked as executable.
If `TEST_PYTHON_EXE` is not user-configured, and thus the above default
is used, a status message is logged by CMake. I've seen this a lot in
other projects, and I like that it shows which values are auto-detected.
However, it's not common in Blender, so if we want we can either remove
it now, or remove it after the buildbot has been set up correctly.
Differential Revision: https://developer.blender.org/D7395
Reviewed by: campbellbarton, mont29, sergey
111 lines
3.8 KiB
Python
111 lines
3.8 KiB
Python
#!/usr/bin/env python3
|
|
# ##### BEGIN GPL LICENSE BLOCK #####
|
|
#
|
|
# This program is free software; you can redistribute it and/or
|
|
# modify it under the terms of the GNU General Public License
|
|
# as published by the Free Software Foundation; either version 2
|
|
# of the License, or (at your option) any later version.
|
|
#
|
|
# This program is distributed in the hope that it will be useful,
|
|
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
# GNU General Public License for more details.
|
|
#
|
|
# You should have received a copy of the GNU General Public License
|
|
# along with this program; if not, write to the Free Software Foundation,
|
|
# Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
|
|
#
|
|
# ##### END GPL LICENSE BLOCK #####
|
|
|
|
# <pep8 compliant>
|
|
|
|
import argparse
|
|
import pathlib
|
|
import sys
|
|
import unittest
|
|
|
|
from modules.test_utils import AbstractBlenderRunnerTest
|
|
|
|
|
|
class AbstractFFmpegTest(AbstractBlenderRunnerTest):
|
|
@classmethod
|
|
def setUpClass(cls):
|
|
cls.blender = args.blender
|
|
cls.testdir = pathlib.Path(args.testdir)
|
|
|
|
|
|
class AbstractFFmpegSequencerTest(AbstractFFmpegTest):
|
|
def get_script_for_file(self, filename: pathlib.Path) -> str:
|
|
movie = self.testdir / filename
|
|
return \
|
|
"import bpy; " \
|
|
"bpy.context.scene.sequence_editor_create(); " \
|
|
"strip = bpy.context.scene.sequence_editor.sequences.new_movie(" \
|
|
"'test_movie', %r, channel=1, frame_start=1); " \
|
|
"print(f'fps:{strip.fps}'); " \
|
|
"print(f'duration:{strip.frame_final_duration}'); " % movie.as_posix()
|
|
|
|
def get_movie_file_field(self, filename: pathlib.Path, field: str) -> str:
|
|
script = self.get_script_for_file(filename)
|
|
output = self.run_blender('', script)
|
|
prefix = field + ":"
|
|
for line in output.splitlines():
|
|
if line.startswith(prefix):
|
|
return line.split(':')[1]
|
|
return ""
|
|
|
|
def get_movie_file_field_float(self, filename: pathlib.Path, field: str) -> float:
|
|
return float(self.get_movie_file_field(filename, field))
|
|
|
|
def get_movie_file_field_int(self, filename: pathlib.Path, field: str) -> float:
|
|
return int(self.get_movie_file_field(filename, field))
|
|
|
|
def get_movie_file_fps(self, filename: pathlib.Path) -> float:
|
|
return self.get_movie_file_field_float(filename, "fps")
|
|
|
|
def get_movie_file_duration(self, filename: pathlib.Path) -> float:
|
|
return self.get_movie_file_field_int(filename, "duration")
|
|
|
|
|
|
class FPSDetectionTest(AbstractFFmpegSequencerTest):
|
|
def test_T51153(self):
|
|
self.assertAlmostEqual(
|
|
self.get_movie_file_fps('T51153_bad_clip_2.mts'),
|
|
29.97,
|
|
places=2)
|
|
|
|
def test_T53857(self):
|
|
self.assertAlmostEqual(
|
|
self.get_movie_file_fps('T53857_2018-01-22_15-30-49.mkv'),
|
|
30.0,
|
|
places=2)
|
|
|
|
def test_T54148(self):
|
|
self.assertAlmostEqual(
|
|
self.get_movie_file_fps('T54148_magn_0.mkv'),
|
|
1.0,
|
|
places=2)
|
|
|
|
def test_T68091(self):
|
|
self.assertAlmostEqual(
|
|
self.get_movie_file_fps('T68091-invalid-nb_frames-at-10fps.mp4'),
|
|
10.0,
|
|
places=2)
|
|
self.assertEqual(
|
|
self.get_movie_file_duration('T68091-invalid-nb_frames-at-10fps.mp4'),
|
|
10)
|
|
|
|
def test_T54834(self):
|
|
self.assertEqual(
|
|
self.get_movie_file_duration('T54834.ogg'),
|
|
50)
|
|
|
|
|
|
if __name__ == '__main__':
|
|
parser = argparse.ArgumentParser()
|
|
parser.add_argument('--blender', required=True)
|
|
parser.add_argument('--testdir', required=True)
|
|
args, remaining = parser.parse_known_args()
|
|
|
|
unittest.main(argv=sys.argv[0:1] + remaining)
|