pyg90alarm 2.5.1__tar.gz → 2.5.2__tar.gz

{pyg90alarm-2.5.1 → pyg90alarm-2.5.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pyg90alarm
-Version: 2.5.1
+Version: 2.5.2
 Summary: G90 Alarm system protocol
 Home-page: https://github.com/hostcc/pyg90alarm
 Author: Ilia Sotnikov

{pyg90alarm-2.5.1 → pyg90alarm-2.5.2}/src/pyg90alarm/local/host_config.py

@@ -35,6 +35,7 @@ class G90SpeechLanguage(IntEnum):
     """
     Supported speech languages.
     """
+    NONE = 0  # Some panels send the value, exact meaning is unknown
     ENGLISH_FEMALE = 1
     ENGLISH_MALE = 2
     CHINESE_FEMALE = 3

{pyg90alarm-2.5.1 → pyg90alarm-2.5.2}/src/pyg90alarm/local/net_config.py

@@ -73,7 +73,7 @@ class G90NetConfig(DataclassLoadSave):
     # Access Point Name (APN) for GPRS connection, as provided by the cellular
     # operator
     apn_name: str = validated_string_field(
-        min_length=1, max_length=100, trust_initial_value=True
+        min_length=0, max_length=100, trust_initial_value=True
     )
     # User name for APN authentication, as provided by the cellular operator
     apn_user: str = validated_string_field(

{pyg90alarm-2.5.1 → pyg90alarm-2.5.2}/src/pyg90alarm.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pyg90alarm
-Version: 2.5.1
+Version: 2.5.2
 Summary: G90 Alarm system protocol
 Home-page: https://github.com/hostcc/pyg90alarm
 Author: Ilia Sotnikov

{pyg90alarm-2.5.1 → pyg90alarm-2.5.2}/tests/test_host_config.py

@@ -164,3 +164,35 @@ async def test_host_config_constraints(
     # Test setting valid value
     setattr(cfg, field_name, valid_value)
     assert getattr(cfg, field_name) == valid_value
+
+
+@pytest.mark.g90device(sent_data=[
+    b'ISTART[106,'
+    b'[900,0,0,1,2,2,60,0,0,60,2]'
+    b']IEND\0',
+    b'ISTARTIEND\0'
+])
+async def test_host_config_speech_language_zero(
+    mock_device: DeviceMock
+) -> None:
+    """
+    Tests for handling host configuration with speech language set to zero.
+    """
+    g90 = G90Alarm(host=mock_device.host, port=mock_device.port)
+
+    # Retrieve configuration
+    cfg = await g90.host_config()
+    assert isinstance(cfg, G90HostConfig)
+
+    # Verify retrieved values
+    assert cfg.speech_language == G90SpeechLanguage.NONE
+
+    # Verify zero value is allowed to be sent back to the panel
+    await cfg.save()
+
+    assert await mock_device.recv_data == [
+        b'ISTART[106,106,""]IEND\0',
+        b'ISTART[107,107,[107,'
+        b'[900,0,0,1,2,2,60,0,0,60,2]'
+        b']]IEND\0'
+    ]

{pyg90alarm-2.5.1 → pyg90alarm-2.5.2}/tests/test_net_config.py

@@ -80,7 +80,7 @@ async def test_net_config(
 @pytest.mark.parametrize(
     'field_name,invalid_value_low,invalid_value_high,valid_value', [
         pytest.param(
-            'apn_name', '', 'a' * 101, 'valid.apn.net',
+            'apn_name', None, 'a' * 101, 'valid.apn.net',
             id='apn_name'
         ),
         pytest.param(
@@ -151,3 +151,42 @@ async def test_net_config_constraints(
     # Test setting valid value
     setattr(cfg, field_name, valid_value)
     assert getattr(cfg, field_name) == valid_value
+
+
+@pytest.mark.g90device(sent_data=[
+    b'ISTART[212,'
+    b'[0,"123456789",1,1,"","user","pwd",3,"54321"]'
+    b']IEND\0',
+    b'ISTARTIEND\0'
+])
+async def test_net_config_apn_name_empty(
+    mock_device: DeviceMock, caplog: pytest.LogCaptureFixture
+) -> None:
+    """
+    Tests for handling network configuration with empty APN name.
+    """
+    g90 = G90Alarm(host=mock_device.host, port=mock_device.port)
+
+    # Retrieve configuration
+    cfg = await g90.net_config()
+    assert isinstance(cfg, G90NetConfig)
+
+    # Verify retrieved values
+    assert cfg.apn_name == ''
+
+    # Ensure that no validation error was logged for empty APN name, the value
+    # of the field as received from the panel is trusted, so no exception will
+    # be raised only error logged
+    assert (
+        'apn_name: Validation failed during initialization for trusted value'
+    ) not in ''.join(caplog.messages)
+
+    # Verify empty APN name is allowed to be sent back to the panel
+    await cfg.save()
+
+    assert await mock_device.recv_data == [
+        b'ISTART[212,212,""]IEND\0',
+        b'ISTART[213,213,[213,'
+        b'[0,"123456789",1,1,"","user","pwd",3]'
+        b']]IEND\0'
+    ]