Skip to content

Commit 70d27e3

Browse files
committed
Revert "fix: make from_dict of PyPDFToDocument more robust (#8422)"
This reverts commit 4e1d6f1.
1 parent 57f43be commit 70d27e3

File tree

3 files changed

+2
-15
lines changed

3 files changed

+2
-15
lines changed

haystack/components/converters/pypdf.py

+2-4
Original file line numberDiff line numberDiff line change
@@ -108,10 +108,8 @@ def from_dict(cls, data):
108108
:returns:
109109
Deserialized component.
110110
"""
111-
# the converter default is `None`, check if it was defined before deserializing
112-
if "converter" in data["init_parameters"]:
113-
converter_class = deserialize_type(data["init_parameters"]["converter"]["type"])
114-
data["init_parameters"]["converter"] = converter_class.from_dict(data["init_parameters"]["converter"])
111+
converter_class = deserialize_type(data["init_parameters"]["converter"]["type"])
112+
data["init_parameters"]["converter"] = converter_class.from_dict(data["init_parameters"]["converter"])
115113
return default_from_dict(cls, data)
116114

117115
@component.output_types(documents=List[Document])

releasenotes/notes/pypdf-converter-robust-from-dict-35ebe6aaab944b19.yaml

-5
This file was deleted.

test/components/converters/test_pypdf_to_document.py

-6
Original file line numberDiff line numberDiff line change
@@ -40,12 +40,6 @@ def test_from_dict(self):
4040
assert isinstance(instance, PyPDFToDocument)
4141
assert isinstance(instance.converter, DefaultConverter)
4242

43-
def test_from_dict_no_converter(self):
44-
data = {"type": "haystack.components.converters.pypdf.PyPDFToDocument", "init_parameters": {}}
45-
instance = PyPDFToDocument.from_dict(data)
46-
assert isinstance(instance, PyPDFToDocument)
47-
assert isinstance(instance.converter, DefaultConverter)
48-
4943
@pytest.mark.integration
5044
def test_run(self, test_files_path, pypdf_converter):
5145
"""

0 commit comments

Comments
 (0)