refactor(players): delete dead VideoPlayer that restarted a unit that does not exist (869f8v8yv)

src/cuemsengine/cues/CueHandler.py

@@ -16,7 +16,6 @@
 from cuemsutils.tools.CTimecode import CTimecode
 
 from ..comms.NodeCommunications import NodeCommunications
-from ..players import VideoPlayer
 from ..players.PlayerHandler import PLAYER_HANDLER
 from ..tools import MtcListener
 from .ActionHandler import ACTION_HANDLER as _ACTION_HANDLER_SINGLETON
@@ -54,8 +53,6 @@ class CueHandler:
     # Instance attributes (declared for IDE/type checker support)
     _armed_cues: list[Cue]
     _armed_cues_set: set[str]
-    _video_players: dict
-    _front_video_player: VideoPlayer | None
     _lock: Lock
     communications_thread: NodeCommunications
 
@@ -65,8 +62,6 @@ def __new__(cls, *args, **kwargs):
             # Initialize instance attributes
             cls._instance._armed_cues = []
             cls._instance._armed_cues_set = set()
-            cls._instance._video_players = {}
-            cls._instance._front_video_player = None
             cls._instance._lock = Lock()
         return cls._instance
 

src/cuemsengine/players/VideoPlayer.py

@@ -2,34 +2,10 @@
 # SPDX-License-Identifier: GPL-3.0-or-later
 # SPDX-FileContributor: Ion Reguera <ion@stagelab.coop>
 # SPDX-FileContributor: Adrià Masip <adria@stagelab.coop>
-from cuemsutils.log import Logger, logged
+from cuemsutils.log import Logger
 
 from ..osc.endpoints import OSC_VIDEOPLAYER_CONF, OSC_VIDEOPLAYER_LAYER_CONF
 from ..osc.OssiaClient import PlayerClient
-from .Player import Player
-
-
-class VideoPlayer(Player):
-    """Video player systemd service wrapper.
-
-    This class restarts the videocomposer service.
-
-    IMPORTANT: This class should not be used, since videocomposer is a systemd
-    service and not a subprocess.
-    """
-
-    def __init__(self):
-        super().__init__()
-        Logger.warning(
-            "Restarting the videocomposer service. Use VideoClient only to"
-            "control videocomposer."
-        )
-
-    @logged
-    def run(self):
-        process_call_list = ["systemctl", "restart", "videocomposer.service"]
-        Logger.info(f"Restarting videocomposer service: {process_call_list}")
-        self.call_subprocess(process_call_list)
 
 
 class VideoClient(PlayerClient):

src/cuemsengine/players/__init__.py

@@ -1,16 +1,16 @@
 # SPDX-FileCopyrightText: 2026 Stagelab Coop SCCL
 # SPDX-License-Identifier: GPL-3.0-or-later
 # SPDX-FileContributor: Adrià Masip <adria@stagelab.coop>
+# SPDX-FileContributor: Ion Reguera <ion@stagelab.coop>
 
 from .AudioPlayer import AudioClient, AudioPlayer
 from .DmxPlayer import DmxClient, DmxPlayer
-from .VideoPlayer import VideoClient, VideoPlayer
+from .VideoPlayer import VideoClient
 
 __all__ = [
     "AudioClient",
     "AudioPlayer",
     "DmxClient",
     "DmxPlayer",
     "VideoClient",
-    "VideoPlayer",
 ]

tests/test_action_cue.py

@@ -61,8 +61,6 @@ def handler():
     h = object.__new__(CueHandler)
     h._armed_cues = []
     h._armed_cues_set = set()
-    h._video_players = {}
-    h._front_video_player = None
     h._lock = __import__("threading").Lock()
     h.communications_thread = MagicMock()
     ACTION_HANDLER.bind_cue_handler(h)

tests/testdev_engines_communication.py

@@ -1,3 +1,7 @@
+# SPDX-FileCopyrightText: 2026 Stagelab Coop SCCL
+# SPDX-License-Identifier: GPL-3.0-or-later
+# SPDX-FileContributor: Ion Reguera <ion@stagelab.coop>
+
 from logging import INFO
 from time import sleep
 from unittest.mock import patch
@@ -82,8 +86,6 @@ def test_two_projects_load_from_controller(
     assert (
         len(mock_player_subprocess) > 0
     ), "Expected player subprocess calls to be recorded"
-    player_types = {call["player"] for call in mock_player_subprocess}
-    assert "VideoPlayer" in player_types, "Expected VideoPlayer to be called"
     # Verify each call has required fields
     for call in mock_player_subprocess:
         assert "player" in call