mirror/python
1
0
Fork 0
mirror of https://github.com/meshtastic/python.git synced 2026-05-18 21:35:07 -04:00
Code Issues Packages Projects Releases Wiki Activity

Fix linting errors and duplicate function

Browse Source
This commit is contained in:
Crank-Git
2025-06-08 19:51:21 -04:00
parent 23be2d2189
commit aa786c7ebd
2 changed files with 48 additions and 36 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
meshtastic/tests/test_node.py
View File

@@ -1487,8 +1487,8 @@ def test_setOwner_whitespace_only_long_name(capsys):
with pytest.raises(SystemExit) as excinfo:
anode.setOwner(long_name=" ")
out, err = capsys.readouterr()
out, _ = capsys.readouterr()
assert "ERROR: Long Name cannot be empty or contain only whitespace characters" in out
assert excinfo.value.code == 1
@@ -1501,8 +1501,8 @@ def test_setOwner_empty_long_name(capsys):
with pytest.raises(SystemExit) as excinfo:
anode.setOwner(long_name="")
out, err = capsys.readouterr()
out, _ = capsys.readouterr()
assert "ERROR: Long Name cannot be empty or contain only whitespace characters" in out
assert excinfo.value.code == 1
@@ -1515,8 +1515,8 @@ def test_setOwner_whitespace_only_short_name(capsys):
with pytest.raises(SystemExit) as excinfo:
anode.setOwner(short_name=" ")
out, err = capsys.readouterr()
out, _ = capsys.readouterr()
assert "ERROR: Short Name cannot be empty or contain only whitespace characters" in out
assert excinfo.value.code == 1
@@ -1529,8 +1529,8 @@ def test_setOwner_empty_short_name(capsys):
with pytest.raises(SystemExit) as excinfo:
anode.setOwner(short_name="")
out, err = capsys.readouterr()
out, _ = capsys.readouterr()
assert "ERROR: Short Name cannot be empty or contain only whitespace characters" in out
assert excinfo.value.code == 1
@@ -1540,10 +1540,10 @@ def test_setOwner_valid_names(caplog):
"""Test setOwner with valid names"""
iface = MagicMock(autospec=MeshInterface)
anode = Node(iface, 123, noProto=True)
with caplog.at_level(logging.DEBUG):
anode.setOwner(long_name="ValidName", short_name="VN")
# Should not raise any exceptions and should call _sendAdmin
iface._sendAdmin.assert_called_once()
assert re.search(r'p.set_owner.long_name:ValidName:', caplog.text, re.MULTILINE)