From e248db6e8d0aad873db5b5ba2823ad9e0801ac1a Mon Sep 17 00:00:00 2001 From: Christopher Albert Date: Tue, 26 Aug 2025 17:51:56 +0200 Subject: [PATCH] fix: test_mpeg_consolidated failure due to unregistered save implementation - Add missing fortplot_animation module import to register save implementation - Change from type-bound anim%save() to procedure save_animation() calls - Type-bound method bypassed facade module registration system - Test now generates valid MPEG files (1619 bytes) with proper structure - Error handling for invalid formats working correctly Fixes #388 --- test/test_mpeg_consolidated.f90 | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/test/test_mpeg_consolidated.f90 b/test/test_mpeg_consolidated.f90 index 30ffae0b..028c034e 100644 --- a/test/test_mpeg_consolidated.f90 +++ b/test/test_mpeg_consolidated.f90 @@ -2,6 +2,7 @@ program test_mpeg_consolidated !! Consolidated MPEG validation test - replaces 20+ redundant MPEG tests !! Covers all essential MPEG functionality with minimal frame counts for speed use fortplot + use fortplot_animation use fortplot_pipe, only: check_ffmpeg_available use fortplot_security, only: safe_remove_file use fortplot_system_runtime, only: is_windows @@ -68,7 +69,7 @@ subroutine test_basic_mpeg_generation_and_validation() ! Create minimal animation (2 frames only for speed) anim = FuncAnimation(update_data, frames=2, interval=200, fig=fig) - call anim%save(test_file, fps=10) + call save_animation(anim, test_file, fps=10) ! Comprehensive validation inquire(file=test_file, exist=file_exists, size=file_size) @@ -139,7 +140,7 @@ subroutine test_error_handling() ! Test invalid format rejection anim = FuncAnimation(dummy_update, frames=1, interval=100, fig=fig) - call anim%save("test.invalid", status=status) + call save_animation(anim, "test.invalid", status=status) if (status == 0) then error stop "ERROR: Should reject invalid format"