diff --git a/testgres/impl/port_manager__generic.py b/testgres/impl/port_manager__generic.py
index a51af2b..452b8df 100755
--- a/testgres/impl/port_manager__generic.py
+++ b/testgres/impl/port_manager__generic.py
@@ -9,6 +9,9 @@
class PortManager__Generic(PortManager):
+ _C_MIN_PORT_NUMBER = 1024
+ _C_MAX_PORT_NUMBER = 65535
+
_os_ops: OsOperations
_guard: object
# TODO: is there better to use bitmap fot _available_ports?
@@ -16,12 +19,21 @@ class PortManager__Generic(PortManager):
_reserved_ports: typing.Set[int]
def __init__(self, os_ops: OsOperations):
+ assert __class__._C_MIN_PORT_NUMBER <= __class__._C_MAX_PORT_NUMBER
+
assert os_ops is not None
assert isinstance(os_ops, OsOperations)
self._os_ops = os_ops
self._guard = threading.Lock()
- self._available_ports: typing.Set[int] = set(range(1024, 65535))
- self._reserved_ports: typing.Set[int] = set()
+
+ self._available_ports = set(
+ range(__class__._C_MIN_PORT_NUMBER, __class__._C_MAX_PORT_NUMBER + 1)
+ )
+ assert len(self._available_ports) == (
+ (__class__._C_MAX_PORT_NUMBER - __class__._C_MIN_PORT_NUMBER) + 1
+ )
+ self._reserved_ports = set()
+ return
def reserve_port(self) -> int:
assert self._guard is not None
@@ -35,9 +47,13 @@ def reserve_port(self) -> int:
t = None
for port in sampled_ports:
+ assert type(port) == int # noqa: E721
assert not (port in self._reserved_ports)
assert port in self._available_ports
+ assert port >= __class__._C_MIN_PORT_NUMBER
+ assert port <= __class__._C_MAX_PORT_NUMBER
+
if not self._os_ops.is_port_free(port):
continue
@@ -51,6 +67,8 @@ def reserve_port(self) -> int:
def release_port(self, number: int) -> None:
assert type(number) == int # noqa: E721
+ assert number >= __class__._C_MIN_PORT_NUMBER
+ assert number <= __class__._C_MAX_PORT_NUMBER
assert self._guard is not None
assert type(self._reserved_ports) == set # noqa: E721
@@ -62,3 +80,4 @@ def release_port(self, number: int) -> None:
self._reserved_ports.discard(number)
assert not (number in self._reserved_ports)
assert number in self._available_ports
+ return
pFad - Phonifier reborn
Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.
Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.
Alternative Proxies:
Alternative Proxy
pFad Proxy
pFad v3 Proxy
pFad v4 Proxy