jyapayne/langflow
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions

fix: refine subflow field filtering to include '|' character check (#4277)

Browse source 
* Refine subflow field filtering to include '|' character check

* Add resource cleanup in test_components_path to ensure proper closure

* Remove unnecessary try-finally block in test_cli.py

* Remove unused pytest fixture decorator from test_components_path function

* Update test duration tracking in backend tests
This commit is contained in:
Gabriel Luiz Freitas Almeida 2024-10-25 10:10:41 -03:00 • committed by GitHub
commit ee101e643c
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 602 additions and 426 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/backend/base/langflow/components/prototypes/sub_flow.py
View file

@ -90,7 +90,7 @@ class SubFlowComponent(Component):
async def generate_results(self) -> list[Data]: async def generate_results(self) -> list[Data]:
tweaks: dict = {} tweaks: dict = {}
for field in self._attributes: for field in self._attributes:
if field != "flow_name": if field != "flow_name" and "|" in field:
[node, name] = field.split("|") [node, name] = field.split("|")
if node not in tweaks: if node not in tweaks:
tweaks[node] = {} tweaks[node] = {}

1024
src/backend/tests/.test_durations
View file

File diff suppressed because it is too large Load diff

2
src/backend/tests/unit/test_cli.py
View file

@ -11,10 +11,10 @@ def default_settings():
] ]
@pytest.mark.usefixtures("client")
def test_components_path(runner, default_settings, tmp_path): def test_components_path(runner, default_settings, tmp_path):
# create a "components" folder # create a "components" folder
temp_dir = tmp_path / "components" temp_dir = tmp_path / "components"
temp_dir.mkdir(exist_ok=True)
result = runner.invoke( result = runner.invoke(
app, app,