Skip to content

Commit d63edf7

Browse files
authored
PYTHON-5524 - Fix CSFLE spec test min version checks (#2516)
1 parent b756bbd commit d63edf7

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

test/asynchronous/unified_format.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,9 @@ async def is_run_on_requirement_satisfied(requirement):
157157
csfle_satisfied = True
158158
req_csfle = requirement.get("csfle")
159159
if req_csfle is True:
160-
min_version_satisfied = Version.from_string("4.2") <= server_version
160+
# Don't overwrite unsatisfied minimum version requirements.
161+
if min_version_satisfied:
162+
min_version_satisfied = Version.from_string("4.2") <= server_version
161163
csfle_satisfied = _HAVE_PYMONGOCRYPT and min_version_satisfied
162164
elif isinstance(req_csfle, dict) and "minLibmongocryptVersion" in req_csfle:
163165
csfle_satisfied = False

test/unified_format.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,9 @@ def is_run_on_requirement_satisfied(requirement):
156156
csfle_satisfied = True
157157
req_csfle = requirement.get("csfle")
158158
if req_csfle is True:
159-
min_version_satisfied = Version.from_string("4.2") <= server_version
159+
# Don't overwrite unsatisfied minimum version requirements.
160+
if min_version_satisfied:
161+
min_version_satisfied = Version.from_string("4.2") <= server_version
160162
csfle_satisfied = _HAVE_PYMONGOCRYPT and min_version_satisfied
161163
elif isinstance(req_csfle, dict) and "minLibmongocryptVersion" in req_csfle:
162164
csfle_satisfied = False

0 commit comments

Comments
 (0)