Add GitHub Actions workflow for unit tests on PRs

Added workflow to run unit tests and doctests on pull requests targeting
main or dev branches, using Python 3.8 container.

The workflow:
- Runs on ubuntu-latest with python:3.8 container
- Installs soco and automate-home dependencies
- Runs unittest discovery for all test files
- Runs doctests on key modules (gateway, message, commands)

Assisted-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: majamassarini

.github/workflows/tests.yml

@@ -0,0 +1,39 @@
+name: Unit Tests
+
+on:
+  pull_request:
+    branches:
+      - main
+      - dev
+  push:
+    branches:
+      - main
+      - dev
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    container:
+      image: python:3.8
+
+    steps:
+    - name: Checkout code
+      uses: actions/checkout@v4
+
+    - name: Install dependencies
+      run: |
+        pip install --upgrade pip
+        pip install soco==0.24.0
+        pip install git+https://github.com/majamassarini/automate-home.git
+
+    - name: Run unit tests
+      run: |
+        python -m unittest discover -v
+
+    - name: Run doctests
+      run: |
+        python -m doctest soco_plugin/gateway.py -v
+        python -m doctest soco_plugin/message.py -v
+        python -m doctest soco_plugin/command/play.py -v
+        python -m doctest soco_plugin/command/pause.py -v
+        python -m doctest soco_plugin/command/stop.py -v

soco_plugin/gateway.py

@@ -159,6 +159,7 @@ def get_action(player: soco.SoCo, msg: "soco_plugin.message.Command") -> Callabl
         >>> import home
         >>> import soco_plugin
         >>> class Player:
+        ...     player_name = "TestPlayer"
         ...     def play(self):
         ...         print("play")
         ...     def pause(self):
@@ -179,7 +180,12 @@ def get_action(player: soco.SoCo, msg: "soco_plugin.message.Command") -> Callabl
         ...         print("play mode is {}".format(mode))
         ...     def get_sonos_playlist_by_attr(self, attr, title):
         ...         print ("playlist title is {}".format(title))
-        ...         return {"uri": "a uri"}
+        ...         class Playlist:
+        ...             def __init__(self):
+        ...                 class Resource:
+        ...                     uri = "a uri"
+        ...                 self.resources = [Resource()]
+        ...         return Playlist()
         ...     def clear_queue(self):
         ...         pass
         ...     def add_uri_to_queue(self, uri):

soco_plugin/tests/test_gateway.py

@@ -29,6 +29,8 @@ async def postpone_gw_running(self):
 
             async def asyncSetUp(self):
                 self._gw = soco_plugin.Gateway()
+                # Make _players dict support iteration for logging
+                self._gw._players = {}
                 self._loop = asyncio.get_event_loop()
                 self._loop.create_task(
                     self._gw.associate_triggers(
@@ -54,6 +56,7 @@ async def test_stopped(self):
 
         test = Test("test_stopped")
         mock = unittest.mock.Mock()
+        mock.player_name = "Bagno"
         event_mock = unittest.mock.Mock()
         event_mock.variables = {"transport_state": "STOPPED"}
         av_mock = unittest.mock.Mock()
@@ -118,6 +121,7 @@ async def test_stopped(self):
 
         test = Test("test_stopped")
         mock = unittest.mock.Mock()
+        mock.player_name = "Bagno"
         event_mock = unittest.mock.Mock()
         event_mock.variables = {"transport_state": "STOPPED"}
         av_mock = unittest.mock.Mock()