github/concrete
1
1
Fork 0
mirror of https://github.com/zama-ai/concrete.git synced 2026-02-08 19:44:57 -05:00
Code Issues Packages Projects Releases Wiki Activity

chore: fix a test which was failing on python 3.9 due to changed python semantics

Browse Source
This commit is contained in:
Umut
2022-08-29 14:43:09 +02:00
parent 05282285a3
commit 847aaae0ab
2 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
concrete/numpy/compilation/configuration.py
View File

@@ -114,17 +114,20 @@ class Configuration:
raise TypeError(f"Unexpected keyword argument '{name}'")
hint = hints[name]
expected = None
is_correctly_typed = True
if name == "insecure_key_cache_location":
if not (value is None or isinstance(value, str)):
is_correctly_typed = False
expected = "Optional[str]"
elif not isinstance(value, hint): # type: ignore
is_correctly_typed = False
if not is_correctly_typed:
expected = hint.__name__ if hasattr(hint, "__name__") else str(hint)
if expected is None:
expected = hint.__name__ if hasattr(hint, "__name__") else str(hint)
raise TypeError(
f"Unexpected type for keyword argument '{name}' "
f"(expected '{expected}', got '{type(value).__name__}')"

2
tests/compilation/test_configuration.py
View File

@@ -72,7 +72,7 @@ def test_configuration_fork():
{"insecure_key_cache_location": 3},
TypeError,
"Unexpected type for keyword argument 'insecure_key_cache_location' "
"(expected 'typing.Union[str, NoneType]', got 'int')",
"(expected 'Optional[str]', got 'int')",
),
],
)