Fix CI

.github/workflows/pythonpackage.yml

@@ -42,7 +42,7 @@ jobs:
           mypy devolo_plc_api
           mypy tests || true
 
-  test38:
+  test_old:
     name: Test with Python 3.8
     runs-on: ubuntu-latest
     steps:
@@ -82,13 +82,13 @@ jobs:
           python -m pip install -e .[test]
       - name: Test with pytest
         run: |
-          pytest --cov=devolo_plc_api
+          pytest --cov=devolo_plc_api --cov-report=xml
       - name: Preserve coverage
         uses: actions/upload-artifact@v4.4.3
         if: matrix.python-version == '3.9'
         with:
           name: coverage
-          path: .coverage
+          path: coverage.xml
 
   coverage:
     name: Upload coverage
@@ -106,11 +106,9 @@ jobs:
         with:
           name: coverage
       - name: Coveralls
-        run: |
-          python -m pip install --upgrade pip
-          python -m pip install wheel coveralls
-          export COVERALLS_REPO_TOKEN=${{ secrets.COVERALLS_TOKEN }}
-          coveralls
+        uses: coverallsapp/github-action@v2.3.4
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
       - name: Clean up coverage
         uses: geekyeggo/delete-artifact@v5.1.0
         with:

devolo_plc_api/__init__.py

@@ -10,4 +10,4 @@
     # package is not installed - e.g. pulled and run locally
     __version__ = "0.0.0"
 
-__all__ = ["Device", "wifi_qr_code", "__version__"]
+__all__ = ["Device", "__version__", "wifi_qr_code"]

devolo_plc_api/device.py

@@ -1,4 +1,5 @@
 """Representation of your devolo device."""
+
 from __future__ import annotations
 
 import asyncio
@@ -227,8 +228,7 @@ async def _get_zeroconf_info(self) -> None:
         )
         while (
             not self._info[DEVICEAPI].properties
-            or not self._info[PLCNETAPI].properties
-            and self.mt_number not in DEVICES_WITHOUT_PLCNET
+            or (not self._info[PLCNETAPI].properties and self.mt_number not in DEVICES_WITHOUT_PLCNET)
         ) and counter < self.MDNS_TIMEOUT:
             counter += 1
             await asyncio.sleep(0.01)

devolo_plc_api/device_api/__init__.py

@@ -28,13 +28,6 @@
 SupportInfoItem = SupportInfoDump.SupportInfoItem
 
 __all__ = [
-    "ConnectedStationInfo",
-    "DeviceApi",
-    "NeighborAPInfo",
-    "RepeatedAPInfo",
-    "SupportInfoItem",
-    "WifiGuestAccessGet",
-    "WifiMultiApGetResponse",
     "CONFIGLAYER_FORMAT",
     "SERVICE_TYPE",
     "UPDATE_AVAILABLE",
@@ -44,4 +37,11 @@
     "WIFI_VAP_GUEST_AP",
     "WIFI_VAP_MAIN_AP",
     "WIFI_VAP_STATION",
+    "ConnectedStationInfo",
+    "DeviceApi",
+    "NeighborAPInfo",
+    "RepeatedAPInfo",
+    "SupportInfoItem",
+    "WifiGuestAccessGet",
+    "WifiMultiApGetResponse",
 ]

devolo_plc_api/plcnet_api/__init__.py

@@ -15,15 +15,15 @@
 LogicalNetwork = GetNetworkOverview.LogicalNetwork
 
 __all__ = [
-    "DataRate",
-    "Device",
-    "LogicalNetwork",
-    "PlcNetApi",
     "DEVICES_WITHOUT_PLCNET",
     "GHN_SPIRIT",
     "HPAV_PANTHER",
     "HPAV_THUNDERBOLT",
     "LOCAL",
     "REMOTE",
     "SERVICE_TYPE",
+    "DataRate",
+    "Device",
+    "LogicalNetwork",
+    "PlcNetApi",
 ]

pyproject.toml

@@ -68,7 +68,7 @@ line-length = 127
 target-version = "py38"
 
 [tool.ruff.lint]
-ignore = ["ANN101", "ANN401", "COM812", "D203", "D205", "D212", "FBT001", "N818"]
+ignore = ["ANN401", "COM812", "D203", "D205", "D212", "FBT001", "N818"]
 select = ["ALL"]
 
 [tool.ruff.lint.isort]