Skip to content

Commit b89b062

Browse files
gijzelaerrclaude
andcommitted
Fix test port conflicts: use unique ports for each test class
- test_server.py: use port 12102 - test_partner.py: use port 12103 This prevents "Address already in use" errors when tests run sequentially. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent e2943d3 commit b89b062

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

tests/test_partner.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,8 @@
1414
class TestPartner(unittest.TestCase):
1515
def setUp(self) -> None:
1616
self.partner = snap7.partner.Partner()
17+
self.partner.port = 12103 # Use unique port for partner tests
18+
self.partner.remote_port = 12103
1719
self.partner.start()
1820

1921
def tearDown(self) -> None:
@@ -48,7 +50,7 @@ def test_get_last_error(self) -> None:
4850
def test_get_param(self) -> None:
4951
expected = (
5052
(Parameter.LocalPort, 0),
51-
(Parameter.RemotePort, 1102), # Non-privileged port
53+
(Parameter.RemotePort, 12103), # Non-privileged port for tests
5254
(Parameter.PingTimeout, 750),
5355
(Parameter.SendTimeout, 10),
5456
(Parameter.RecvTimeout, 3000),

tests/test_server.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
class TestServer(unittest.TestCase):
1717
def setUp(self) -> None:
1818
self.server = Server()
19-
self.server.start(tcp_port=1102)
19+
self.server.start(tcp_port=12102) # Use unique port for server tests
2020

2121
def tearDown(self) -> None:
2222
self.server.stop()
@@ -120,7 +120,7 @@ def test_start_to(self) -> None:
120120

121121
def test_get_param(self) -> None:
122122
# check the defaults
123-
self.assertEqual(self.server.get_param(Parameter.LocalPort), 1102)
123+
self.assertEqual(self.server.get_param(Parameter.LocalPort), 12102)
124124
self.assertEqual(self.server.get_param(Parameter.WorkInterval), 100)
125125
self.assertEqual(self.server.get_param(Parameter.MaxClients), 1024)
126126

0 commit comments

Comments
 (0)