May 6, 2025 · May 7, 2025 · May 7, 2025 · May 7, 2025 · May 7, 2025 · May 7, 2025
diff --git a/testgres/impl/port_manager__generic.py b/testgres/impl/port_manager__generic.py


 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?
    _lock_dir: str

    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 = set(range(1024, 65536))

        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()
        self._lock_dir = None

                assert not (port in self._reserved_ports)
                assert port in self._available_ports

                assert port >=0
                assert port <=65535
                assert port >=__class__.C_MIN_PORT_NUMBER
                assert port <=__class__.C_MAX_PORT_NUMBER

                if not self._os_ops.is_port_free(port):
                    continue

    def release_port(self, number: int) -> None:
        assert type(number) == int  # noqa: E721
        assert number >=0
        assert number <=65535
        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
Original file line number	Diff line number	Diff line change
Expand Up		@@ -11,6 +11,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?
Expand All		@@ -20,11 +23,20 @@ class PortManager__Generic(PortManager):
		_lock_dir: str

		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 = set(range(1024, 65536))

		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()
		self._lock_dir = None

Expand DownExpand Up		@@ -56,8 +68,8 @@ def reserve_port(self) -> int:
		assert not (port in self._reserved_ports)
		assert port in self._available_ports

		assert port >=0
		assert port <=65535
		assert port >=__class__.C_MIN_PORT_NUMBER
		assert port <=__class__.C_MAX_PORT_NUMBER

		if not self._os_ops.is_port_free(port):
		continue
Expand DownExpand Up		@@ -86,8 +98,8 @@ def reserve_port(self) -> int:

		def release_port(self, number: int) -> None:
		assert type(number) == int # noqa: E721
		assert number >=0
		assert number <=65535
		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
Expand Down