NAPALM/napalm-sonos
1
0 Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

chore: adopt the NAPALM driver standard #6

Merged
christianmanivong merged 4 commits from chore/driver-standard into main 2026-10-23 14:42:36 +00:00
Conversation 0 Commits 4 Files Changed 11
+9 -6
2 changed files with 9 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit ad69c0e643 - Show all commits

No files matched your search

napalm_sonos/sonos_driver.py
+1 -5
View File
@@ -4,7 +4,6 @@ import socket
from typing import Any from typing import Any
from napalm.base.exceptions import ConnectionException from napalm.base.exceptions import ConnectionException
from napalm_device_types import FingerprintRule, MediaDriver, PortSpec from napalm_device_types import FingerprintRule, MediaDriver, PortSpec
"""NAPALM driver for Sonos speakers. """NAPALM driver for Sonos speakers.
@@ -15,7 +14,6 @@ media-playback controls.
""" """
class SonosDriver(MediaDriver): class SonosDriver(MediaDriver):
"""NAPALM driver for controlling Sonos speakers. """NAPALM driver for controlling Sonos speakers.
@@ -74,9 +72,7 @@ class SonosDriver(MediaDriver):
if self._device is None: if self._device is None:
return {"is_alive": False} return {"is_alive": False}
try: try:
sock = socket.create_connection( sock = socket.create_connection((self.hostname, 1400), timeout=5)
(self.hostname, 1400), timeout=5
)
sock.close() sock.close()
return {"is_alive": True} return {"is_alive": True}
except OSError: except OSError:
pyproject.toml
+8 -1
View File
@@ -48,7 +48,14 @@ include = ["napalm_sonos*"]
[tool.ruff] [tool.ruff]
line-length = 100 line-length = 100
target-version = "py39" target-version = "py310"
[tool.ruff.lint]
select = ["E", "F", "I", "UP", "B"]
[tool.ruff.lint.per-file-ignores]
# Recorded device output keeps its lines.
"tests/**" = ["E501"]
[tool.pytest.ini_options] [tool.pytest.ini_options]
testpaths = ["tests"] testpaths = ["tests"]