Added code to raise exception with string - now reformatted for appro…

…val (#1952) Reformat files for code format check
Zulko · May 26, 2023 · b5bb086 · b5bb086
1 parent 57c279a
commit b5bb086
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/moviepy/video/VideoClip.py b/moviepy/video/VideoClip.py
@@ -1190,6 +1190,10 @@ def __init__(self, size, color=None, is_mask=False, duration=None):
                 color = (0, 0, 0)
             elif not hasattr(color, "__getitem__"):
                 raise Exception("Color has to contain RGB of the clip")
+            elif isinstance(color, str):
+                raise Exception(
+                    "Color cannot be string. Color has to contain RGB of the clip"
+                )
             shape = (h, w, len(color))
 
         super().__init__(

diff --git a/tests/test_VideoClip.py b/tests/test_VideoClip.py
@@ -264,7 +264,10 @@ def test_oncolor(util):
         clip = ColorClip(size=(100, 60), color=(255, 0, 0), is_mask=True)
 
     with pytest.raises(Exception):
-        clip = ColorClip(size=(100, 60), color=0.4, is_mask=False)
+        clip = ColorClip(size=(100, 60), color=0.4, ismask=False)
+
+    with pytest.raises(Exception):
+        clip = ColorClip(size=(100, 60), color="black")
 
 
 def test_setaudio(util):