Merge pull request #42 from homeassistant-projects/test-fixes

Attempting to fix workflows and unit tests

Author: EVWorth

.github/workflows/python-package.yml

@@ -25,4 +25,4 @@ jobs:
 
       - name: Test with pytest
         run: |
-          uv run --frozen pytest
+          uv run --frozen pytest --isolate

.gitignore

@@ -2,7 +2,7 @@
 *.pyc
 *.log
 *.egg-info
-
+.devcontainer
 dist
 build
 

pyproject.toml

@@ -32,6 +32,7 @@ dev = [
     "pre-commit>=4.2.0",
     "pytest>=8.4.1",
     "pytest-asyncio>=1.0.0",
+    "pytest-isolate>=0.0.12",
     "pytest-mock>=3.14.1",
     "ruff>=0.12.3",
     "twine>=6.1.0",

tests/conftest.py

@@ -21,7 +21,7 @@
 
 # Modify sys.path to include the project root
 sys.path.insert(0, str(project_root))
-test_file_dir = project_root / "tests" / "data_files"
+test_file_dir = project_root / "data_files"
 # pylint: disable=wrong-import-position
 # ruff: noqa: E402
 # flake8: noqa: E402

tests/test_gateway.py

@@ -126,41 +126,6 @@ def test_set_gateway_attributes_sets_attributes_correctly():
     assert gateway.cellular_connection_signal_strength == 4.5
 
 
-# backoff object can be created with default values and current backoff interval can be retrieved
-def test_default_values2():
-    """
-    Test that ADTPulseGateway object can be created with default values
-    """
-    gateway = ADTPulseGateway()
-    assert gateway.manufacturer == "Unknown"
-    assert gateway._status_text == "OFFLINE"
-    assert gateway.backoff.name == "Gateway"
-    assert gateway.backoff.initial_backoff_interval == ADT_DEFAULT_POLL_INTERVAL
-    assert (
-        gateway.backoff._max_backoff_interval == ADT_GATEWAY_MAX_OFFLINE_POLL_INTERVAL
-    )
-    assert gateway.backoff.backoff_count == 0
-    assert gateway.backoff.expiration_time == 0.0
-    assert gateway.backoff._detailed_debug_logging is False
-    assert gateway.backoff._threshold == 0
-    assert gateway.model is None
-    assert gateway.serial_number is None
-    assert gateway.next_update == 0
-    assert gateway.last_update == 0
-    assert gateway.firmware_version is None
-    assert gateway.hardware_version is None
-    assert gateway.primary_connection_type is None
-    assert gateway.broadband_connection_status is None
-    assert gateway.cellular_connection_status is None
-    assert gateway._cellular_connection_signal_strength == 0.0
-    assert gateway.broadband_lan_ip_address is None
-    assert gateway._broadband_lan_mac is None
-    assert gateway.device_lan_ip_address is None
-    assert gateway._device_lan_mac is None
-    assert gateway.router_lan_ip_address is None
-    assert gateway.router_wan_ip_address is None
-
-
 # backoff object can be incremented and reset correctly
 def test_backoff_increment_and_reset():
     """