More edge case testing
This commit is contained in:
@@ -214,6 +214,12 @@ def merge_video(target_path : str, temp_video_fps : Fps, output_video_resolution
|
|||||||
if output_file_format in [ 'm4v', 'mkv', 'mp4' ] and output_video_encoder == 'rawvideo':
|
if output_file_format in [ 'm4v', 'mkv', 'mp4' ] and output_video_encoder == 'rawvideo':
|
||||||
output_video_encoder = 'libx264'
|
output_video_encoder = 'libx264'
|
||||||
|
|
||||||
|
if output_file_format == 'm4v':
|
||||||
|
output_video_encoder = 'libx264'
|
||||||
|
|
||||||
|
if output_file_format == 'mov' and output_video_encoder == 'libvpx-vp9':
|
||||||
|
output_video_encoder = 'libx264'
|
||||||
|
|
||||||
if output_file_format == 'webm':
|
if output_file_format == 'webm':
|
||||||
output_video_encoder = 'libvpx-vp9'
|
output_video_encoder = 'libvpx-vp9'
|
||||||
|
|
||||||
|
|||||||
@@ -97,7 +97,7 @@ def test_merge_video() -> None:
|
|||||||
create_temp_directory(target_path)
|
create_temp_directory(target_path)
|
||||||
extract_frames(target_path, '452x240', 25.0, 0, 1)
|
extract_frames(target_path, '452x240', 25.0, 0, 1)
|
||||||
|
|
||||||
assert merge_video(target_path, 25.0, '452x240', 25.0, 0, 1) is True
|
assert merge_video(target_path, 25.0, '452x240', 25.0, 0, 1) is True
|
||||||
|
|
||||||
clear_temp_directory(target_path)
|
clear_temp_directory(target_path)
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user