Add tests for logos and deleting

Author: JonnyWong16

tests/test_mixins.py

@@ -219,6 +219,10 @@ def lock_art(obj):
     _test_mixins_lock_image(obj, "arts")
 
 
+def lock_logo(obj):
+    _test_mixins_lock_image(obj, "logos")
+
+
 def lock_poster(obj):
     _test_mixins_lock_image(obj, "posters")
 
@@ -228,6 +232,7 @@ def _test_mixins_edit_image(obj, attr):
     get_img_method = getattr(obj, attr)
     set_img_method = getattr(obj, "set" + cap_attr)
     upload_img_method = getattr(obj, "upload" + cap_attr)
+    delete_img_method = getattr(obj, "delete" + cap_attr)
     images = get_img_method()
     if images:
         default_image = images[0]
@@ -270,6 +275,12 @@ def _test_mixins_edit_image(obj, attr):
         ]
         assert file_image
 
+    # Test delete image
+    delete_img_method()
+    images = get_img_method()
+    selected_image = next((i for i in images if i.selected), None)
+    assert selected_image is None
+
     # Reset to default image
     if default_image:
         set_img_method(default_image)
@@ -283,6 +294,10 @@ def edit_art(obj):
     _test_mixins_edit_image(obj, "arts")
 
 
+def edit_logo(obj):
+    _test_mixins_edit_image(obj, "logos")
+
+
 def edit_poster(obj):
     _test_mixins_edit_image(obj, "posters")
 
@@ -330,9 +345,15 @@ def _test_mixins_edit_theme(obj):
     obj.lockTheme()
     obj.reload()
     assert "theme" in _fields()
+
+    # Set the theme
     with pytest.raises(NotImplementedError):
         obj.setTheme(themes[0])
 
+    # Delete the theme
+    with pytest.raises(NotImplementedError):
+        obj.deleteTheme()
+
 
 def edit_theme(obj):
     _test_mixins_edit_theme(obj)

tests/test_video.py

@@ -694,8 +694,10 @@ def test_video_Movie_mixins_edit_advanced_settings(movie):
 @pytest.mark.xfail(reason="Changing images fails randomly")
 def test_video_Movie_mixins_images(movie):
     test_mixins.lock_art(movie)
+    test_mixins.lock_logo(movie)
     test_mixins.lock_poster(movie)
     test_mixins.edit_art(movie)
+    test_mixins.edit_logo(movie)
     test_mixins.edit_poster(movie)
 
 
@@ -961,8 +963,10 @@ def test_video_Show_mixins_edit_advanced_settings(show):
 @pytest.mark.xfail(reason="Changing images fails randomly")
 def test_video_Show_mixins_images(show):
     test_mixins.lock_art(show)
+    test_mixins.lock_logo(show)
     test_mixins.lock_poster(show)
     test_mixins.edit_art(show)
+    test_mixins.edit_logo(show)
     test_mixins.edit_poster(show)
     test_mixins.attr_artUrl(show)
     test_mixins.attr_posterUrl(show)
@@ -1118,8 +1122,10 @@ def test_video_Season_episodes(show):
 def test_video_Season_mixins_images(show):
     season = show.season(season=1)
     test_mixins.lock_art(season)
+    test_mixins.lock_logo(season)
     test_mixins.lock_poster(season)
     test_mixins.edit_art(season)
+    test_mixins.edit_logo(season)
     test_mixins.edit_poster(season)
     test_mixins.attr_artUrl(season)
     test_mixins.attr_posterUrl(season)
@@ -1336,8 +1342,10 @@ def test_video_Episode_unwatched(tvshows):
 @pytest.mark.xfail(reason="Changing images fails randomly")
 def test_video_Episode_mixins_images(episode):
     test_mixins.lock_art(episode)
+    test_mixins.lock_logo(episode)
     test_mixins.lock_poster(episode)
-    # test_mixins.edit_art(episode)  # Uploading episode artwork is broken in Plex
+    test_mixins.edit_art(episode)
+    test_mixins.edit_logo(episode)
     test_mixins.edit_poster(episode)
     test_mixins.attr_artUrl(episode)
     test_mixins.attr_posterUrl(episode)