Add more tests

[tburrows13]

The test coverage is currently at 67%. It would be good to be able to automatically test more things in the test suite. You can see which files are covered at https://coveralls.io/github/Zulko/moviepy (or PyCharm has a test coverage plugin that can be enabled).

In particular:

• some effects in moviepy/video/fx are not tested at all in their respective test function in test_fx.py.
• video/compositing/transitions.py
• audio/fx/audio_fadein and audio_fadeout

For the video tests it is better to use BitmapClip because it allows you to test pixel-perfect effects. But generally just testing that an action can be performed without crashing is better than nothing.

[mondeja]

FX tests checklist

💯 means 100% coverage

Video / audio

• accel_decel
• blackwhite [Test properly 'blackwhite' FX using 'BitmapClip' #1430] 💯
• blink
• multiply_color [Improve VideoClip testing by inspecting numpy frame arrays #1124]
• crop [Improve VideoClip testing by inspecting numpy frame arrays #1124]
• even_size [Improve VideoClip testing by inspecting numpy frame arrays #1124]
• fadein (more duration arguments) [Tests for loop using BitmapClip #1368 #1370] 💯
• fadeout (rewrite using BitmapClip) 💯
• freeze [Improve VideoClip testing by inspecting numpy frame arrays #1124, Fix 'freeze' FX not freezing at the end and add proper tests #1461] 💯
• freeze_region (mask argument) [Improve VideoClip testing by inspecting numpy frame arrays #1124]
• gamma_corr
• headblur
• invert_colors [Improve VideoClip testing by inspecting numpy frame arrays #1124] 💯
• loop (use BitmapClip testing all behaviours) [Tests for loop using BitmapClip #1368 #1370] 💯
• lum_contrast (rewrite using BitmapClip) 💯
• make_loopable 💯
• margin (test opacity argument) [Improve VideoClip testing by inspecting numpy frame arrays #1124, Test left, right, top, bottom and color arguments of margin FX #1579]
• mask_and [Add 'mask_and' FX tests #1486] 💯
• mask_color
• mask_or [Add 'mask_or' FX tests #1485] 💯
• mirror_x [Improve VideoClip testing by inspecting numpy frame arrays #1124] 💯
• mirror_y [Improve VideoClip testing by inspecting numpy frame arrays #1124] 💯
• multiply_color [Improve VideoClip testing by inspecting numpy frame arrays #1124] 💯
• multiply_speed [Improve VideoClip testing by inspecting numpy frame arrays #1124] 💯
• painting
• resize [Test 'resize' FX against all implementations and arguments #1495]
• rotate [Improve VideoClip testing by inspecting numpy frame arrays #1124, Test all 'rotate' FX arguments combinations #1484, Test mask rotation #1543, Add tests for rotate FX behaviour when Pillow is not installed #1544, Test 'rotate' FX supported arguments by PIL version #1545] 💯
• scroll
• supersample
• time_mirror [Improve VideoClip testing by inspecting numpy frame arrays #1124] 💯
• time_symmetrize [Improve VideoClip testing by inspecting numpy frame arrays #1124] 💯

Audio

• audio_fadein [Add tests for 'audio_fadein' FX #1574] 💯
• audio_fadeout [Add tests for 'audio_fadeout' FX #1578] 💯
• audio_delay [Add 'audio_delay' FX #1481] 💯
• audio_normalize [Add audio normalization function #609, Fix audio normalizing for mute clips #1401] 💯
• multiply_stereo_volume [Add 'multiply_stereo_volume' FX; rename 'volumex' to 'multiply_volume' #1424] 💯
• multiply_volume [Add 'multiply_volume' FX tests #1480] 💯

---

Video tools tests checklist

💯 means 100% coverage

• credits.CreditsClip 💯
• cuts.find_video_period 💯
• cuts.FramesMatch [Add tests for 'FramesMatches' #1588] 💯
• cuts.FramesMatches [Add tests for 'FramesMatches' #1588, Add more tests and documentation for FramesMatches #1600]
• cuts.detect_scenes (test with luminosities != None)
• drawing.blit 💯
• drawing.color_gradient [Add tests for drawing video tools #1602] 💯
• drawing.color_split [Add tests for drawing video tools #1602] 💯
• drawing.circle [Add tests for drawing video tools #1602] 💯
• interpolators.Interpolator [Add tests and documentation for interpolators #1606] 💯
• segmenting.find_objects [Add tests for 'find_objects' video tool #1607] 💯
• subtitles.SubtitlesClip
• subtitles.file_to_subtitles 💯
• tracking.manual_tracking
• tracking.findAround
• tracking.autoTrack

[tburrows13]

It would be quite hard to write BitmapClip tests for all of the effects, I think. Although you did manage with blackwhite in #1430 I wouldn't say it is worth it for some of the more 'color-gradienty' ones (such as headblur and painting).

[mondeja]

It would be quite hard to write BitmapClip tests for all of the effects...

I have found it much easier to use ColorClip in tests for mask_or and mask_and testing ImageClip and np.ndarray types as arguments.

[tburrows13]

Closing as these checklists are very out-of-date now that 2.0 is out. Test coverage is now at 86% 💪 (possibly due to obsolete code being removed as much as tests being added... but hey!)