chore: modernize development infrastructure and build configuration

.github/workflows/ci.yml

@@ -1,42 +1,77 @@
 name: CI
 
 on:
-  # Run action when pushed to master, or for commits in a pull request.
   push:
-    branches:
-      - master
+    branches: [main, master]
   pull_request:
-    branches:
-      - master
+    branches: [main, master]
 
 defaults:
   run:
     shell: bash
 
 jobs:
-  checks:
-    name: Code checks
-    runs-on: ${{ matrix.os }}
+  build-and-test:
+    runs-on: ubuntu-latest
     strategy:
       fail-fast: false
       matrix:
-        os: [ ubuntu-latest ]
         python-version: [ "3.10", "3.11", "3.12", "3.13" ]
     steps:
-      - name: Check out ${{ github.sha }} from repository ${{ github.repository }}
+      - name: Check out code
         uses: actions/checkout@v4
 
-      - name: Install poetry
-        run: pipx install poetry
+      - name: Show GitHub context variables
+        run: |
+          echo "GITHUB_SHA: $GITHUB_SHA"
+          echo "GITHUB_REPOSITORY: $GITHUB_REPOSITORY"
 
       - name: Set up Python ${{ matrix.python-version }}
         uses: actions/setup-python@v5
         with:
           python-version: ${{ matrix.python-version }}
-          cache: 'poetry'
+          # ensure activation of poetry cache after poetry installation
 
-      - name: Install dependencies
-        run: poetry install --no-interaction
+      - name: Install Poetry
+        run: |
+          python -m pip install --upgrade pip
+          pip install poetry
 
-      - name: Run checks
-        run: make check
+      - name: Configure Poetry
+        run: |
+          poetry config virtualenvs.create true
+          poetry config virtualenvs.in-project true
+
+      - name: Cache Poetry dependencies
+        uses: actions/cache@v4
+        with:
+          path: ~/.cache/pypoetry
+          key: ${{ runner.os }}-poetry-${{ hashFiles('**/poetry.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-poetry-
+
+      - name: Install dependencies (with dev)
+        run: poetry install --with dev --no-interaction
+
+      - name: Run code quality checks
+        run: |
+          echo "Running pylint..."
+          make check-pylint
+          echo "Running black format check..."
+          make check-black
+
+      - name: Run tests with coverage (if tests exist)
+        run: |
+          if [ -d "tests" ] && find tests -name "test_*.py" | grep -q .; then
+            echo "Running tests..."
+            make test
+          else
+            echo "No test files found, skipping tests"
+          fi
+
+      - name: Upload coverage to Codecov (optional)
+        if: matrix.python-version == '3.12'
+        uses: codecov/codecov-action@v4
+        with:
+          file: ./coverage.xml
+          fail_ci_if_error: false

Makefile

@@ -1,7 +1,7 @@
 check: check-pylint check-black
 
 check-pylint:
-	@poetry run pylint aiocomfoconnect/*.py
+	@poetry run pylint --load-plugins=pylint_protobuf aiocomfoconnect/*.py
 
 check-black:
 	@poetry run black --check aiocomfoconnect/*.py
@@ -11,7 +11,7 @@ codefix:
 	@poetry run black aiocomfoconnect/*.py
 
 test:
-	@poetry run pytest
+	@poetry run pytest --cov=aiocomfoconnect --cov-report=term --cov-report=xml
 
 build:
 	docker build -t aiocomfoconnect .

aiocomfoconnect/discovery.py

@@ -38,12 +38,12 @@ def connection_made(self, transport: asyncio.transports.DatagramTransport):
             # Determine broadcast address programmatically
             try:
                 gws = netifaces.gateways()
-                default_iface = gws['default'][netifaces.AF_INET][1]
+                default_iface = gws["default"][netifaces.AF_INET][1]
                 addrs = netifaces.ifaddresses(default_iface)
-                broadcast_addr = addrs[netifaces.AF_INET][0].get('broadcast', '255.255.255.255')
-            except Exception as e:
+                broadcast_addr = addrs[netifaces.AF_INET][0].get("broadcast", "255.255.255.255")
+            except (KeyError, ValueError, OSError) as e:
                 _LOGGER.warning("Could not determine broadcast address, using 255.255.255.255: %s", e)
-                broadcast_addr = '255.255.255.255'
+                broadcast_addr = "255.255.255.255"
             _LOGGER.debug("Sending discovery request to broadcast:%d (%s)", Bridge.PORT, broadcast_addr)
             self.transport.sendto(b"\x0a\x00", (broadcast_addr, Bridge.PORT))
 
@@ -60,7 +60,7 @@ def datagram_received(self, data: Union[bytes, str], addr: tuple[str | Any, int]
             parser.ParseFromString(data)
 
             self._bridges.append(Bridge(host=parser.searchGatewayResponse.ipaddress, uuid=parser.searchGatewayResponse.uuid.hex()))
-        except Exception as exc:
+        except (ValueError, AttributeError, TypeError) as exc:
             _LOGGER.error("Failed to parse discovery response from %s: %s", addr, exc)
             return