Skip to content

Commit fa72ed1

Browse files
committed
Fix Output return value from imported_baseline - see HEA-784
1 parent 58368ee commit fa72ed1

File tree

2 files changed

+4
-9
lines changed

2 files changed

+4
-9
lines changed

pipelines/assets/base.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -314,7 +314,7 @@ def get_bss_dataframe(
314314
end_strings: Optional[list[str]] = None,
315315
header_rows: list[int] = [3, 4, 5], # List of row indexes that contain the Wealth Group and other headers
316316
num_summary_cols: Optional[int] = None,
317-
) -> pd.DataFrame:
317+
) -> Output[pd.DataFrame]:
318318
"""
319319
Retrieve a worksheet from a BSS and return it as a DataFrame.
320320

pipelines/assets/fixtures.py

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030

3131
def validate_instances(
3232
context: AssetExecutionContext, config: BSSMetadataConfig, instances: dict[str, list[dict]], partition_key: str
33-
) -> tuple[dict[str, list[dict]], dict]:
33+
) -> Output[dict[str, list[dict]]]:
3434
"""
3535
Validate the instances for a set of related models, prior to loading them as a fixture.
3636
@@ -322,7 +322,7 @@ def get_fixture_from_instances(instance_dict: dict[str, list[dict]]) -> tuple[li
322322
return Output(fixture, metadata=metadata)
323323

324324

325-
def import_fixture(fixture: list[dict]) -> dict:
325+
def import_fixture(fixture: list[dict]) -> Output[None]:
326326
"""
327327
Import a Django fixture and return a metadata dictionary.
328328
"""
@@ -436,9 +436,4 @@ def imported_baseline(
436436
"""
437437
Imported Django fixture for a BSS, added to the Django database.
438438
"""
439-
metadata = import_fixture(consolidated_fixture)
440-
441-
return Output(
442-
None,
443-
metadata=metadata,
444-
)
439+
return import_fixture(consolidated_fixture)

0 commit comments

Comments
 (0)