Refactor docstrings for clarity and enhance test descriptions for payload handling

Author: iMicknl

pyoverkiz/client.py

@@ -637,13 +637,11 @@ async def execute_action_group(
         mode: CommandMode | None = None,
         label: str | None = "python-overkiz-api",
     ) -> str:
-        """
-        Execute a non-persistent action group
+        """Execute a non-persistent action group.
 
         The executed action group does not have to be persisted on the server before use.
         Per-session rate-limit : 1 calls per 28min 48s period for all operations of the same category (exec)
         """
-
         """Send several commands in one call"""
 
         # Build a logical (snake_case) payload using model helpers and convert it

tests/test_models.py

@@ -303,6 +303,7 @@ def test_bad_list_value(self):
 
 
 def test_command_to_payload_omits_none():
+    """Command.to_payload omits None fields from the resulting payload."""
     from pyoverkiz.enums.command import OverkizCommand
     from pyoverkiz.models import Command
 
@@ -313,6 +314,7 @@ def test_command_to_payload_omits_none():
 
 
 def test_action_to_payload_and_parameters_conversion():
+    """Action.to_payload converts nested Command enums to primitives."""
     from pyoverkiz.enums.command import OverkizCommand, OverkizCommandParam
     from pyoverkiz.models import Action, Command
 

tests/test_serializers.py

@@ -1,9 +1,12 @@
+"""Tests for pyoverkiz.serializers."""
+
 from __future__ import annotations
 
 from pyoverkiz.serializers import prepare_payload
 
 
 def test_prepare_payload_camelizes_and_fixes_device_url():
+    """Test that prepare_payload converts snake_case to camelCase and fixes abbreviations."""
     payload = {
         "label": "test",
         "actions": [{"device_url": "rts://1/2", "commands": [{"name": "close"}]}],
@@ -17,6 +20,7 @@ def test_prepare_payload_camelizes_and_fixes_device_url():
 
 
 def test_prepare_payload_nested_lists_and_dicts():
+    """Test that prepare_payload handles nested lists and dicts correctly."""
     payload = {
         "actions": [
             {