#554: Fixed bucketfs ports (#555)

fixes #554
---------

Co-authored-by: Torsten Kilias <[email protected]>

Author: tomuben

doc/changes/changelog.md

@@ -1,6 +1,7 @@
 # Changes
 
 * [unreleased](unreleased.md)
+* [4.4.1](changes_4.4.1.md)
 * [4.4.0](changes_4.4.0.md)
 * [4.3.0](changes_4.3.0.md)
 * [4.2.0](changes_4.2.0.md)
@@ -70,6 +71,7 @@ changes_0.3.0
 changes_0.2.0
 changes_0.1.0
 unreleased
+changes_4.4.1
 changes_4.4.0
 changes_4.3.0
 changes_4.2.0

doc/changes/changes_4.4.1.md

@@ -0,0 +1,8 @@
+# 4.4.1 - 2025-11-12
+
+This hotfix release fixes a bug with the BucketFS ports, in which the wrong Http port for the deprecated `bucketfs` property was returned.
+
+## Bugs
+
+ - 542: Removed Publishing Docker image from workflow
+ - #554: Fixed bucketfs ports

doc/changes/unreleased.md

@@ -1,4 +1 @@
 # Unreleased
-
-## Bugs
- - 542: Removed Publishing Docker image from workflow

exasol_integration_test_docker_environment/lib/test_environment/ports.py

@@ -1,5 +1,6 @@
 import logging
 import socket
+import warnings
 from collections.abc import Generator
 from contextlib import ExitStack
 from typing import (
@@ -81,27 +82,27 @@ def __init__(
 
     @property
     def bucketfs(self) -> int:
-        logging.warn(
+        warnings.warn(
             f"Property Ports.bucketfs is deprecated, use Ports.bucketfs_http instead.",
             DeprecationWarning,
         )
-        return self._bucketfs_https
+        return self._bucketfs_http
 
     @bucketfs.setter
     def bucketfs(self, value: int) -> None:
-        logging.warn(
+        warnings.warn(
             f"Property Ports.bucketfs is deprecated, use Ports.bucketfs_http instead.",
             DeprecationWarning,
         )
-        self._bucketfs_https = value
+        self._bucketfs_http = value
 
     @property
     def bucketfs_http(self) -> int:
         return self._bucketfs_http
 
     @bucketfs_http.setter
     def bucketfs_http(self, value: int) -> None:
-        logging.warn(
+        warnings.warn(
             f"Setting values of class Ports after instantiation is deprecated.",
             DeprecationWarning,
         )
@@ -113,7 +114,7 @@ def bucketfs_https(self) -> int:
 
     @bucketfs_https.setter
     def bucketfs_https(self, value: int) -> None:
-        logging.warn(
+        warnings.warn(
             f"Setting values of class Ports after instantiation is deprecated.",
             DeprecationWarning,
         )
@@ -125,7 +126,7 @@ def database(self) -> int:
 
     @database.setter
     def database(self, value: int) -> None:
-        logging.warn(
+        warnings.warn(
             f"Setting values of class Ports after instantiation is deprecated.",
             DeprecationWarning,
         )
@@ -137,7 +138,7 @@ def ssh(self) -> Optional[int]:
 
     @ssh.setter
     def ssh(self, value: Optional[int]) -> None:
-        logging.warn(
+        warnings.warn(
             f"Setting values of class Ports after instantiation is deprecated.",
             DeprecationWarning,
         )

exasol_integration_test_docker_environment/version.py

@@ -10,6 +10,6 @@
 
 MAJOR = 4
 MINOR = 4
-PATCH = 0
+PATCH = 1
 VERSION = f"{MAJOR}.{MINOR}.{PATCH}"
 __version__ = VERSION

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "exasol-integration-test-docker-environment"
-version = "4.4.0"
+version = "4.4.1"
 description = "Integration Test Docker Environment for Exasol"
 authors = [
     {name="Torsten Kilias", email="[email protected]"},

test/unit/test_ports.py

@@ -0,0 +1,28 @@
+from exasol_integration_test_docker_environment.lib.test_environment.ports import Ports
+
+
+def test_forward_ports():
+    p = Ports.forward
+    assert p.database == 8563
+    assert p.bucketfs == 2580
+    assert p.bucketfs_http == 2580
+    assert p.ssh == 20002
+    assert p.bucketfs_https == 2581
+
+
+def test_default_ports():
+    p = Ports.default_ports
+    assert p.database == 8563
+    assert p.bucketfs == 2580
+    assert p.bucketfs_http == 2580
+    assert p.ssh == 22
+    assert p.bucketfs_https == 2581
+
+
+def test_external_ports():
+    p = Ports.external
+    assert p.database == 8563
+    assert p.bucketfs == 2580
+    assert p.bucketfs_http == 2580
+    assert p.ssh is None
+    assert p.bucketfs_https == 2581