Skip to content

Commit cdf3479

Browse files
committed
pass no_empty
1 parent 6ee92f0 commit cdf3479

2 files changed

Lines changed: 19 additions & 1 deletion

File tree

src/hdx/data/dataset.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2506,8 +2506,9 @@ def process_row(row: Sequence | Mapping) -> Sequence | Mapping | None:
25062506
format=format,
25072507
encoding=encoding,
25082508
row_function=process_row,
2509+
no_empty=no_empty,
25092510
)
2510-
if not rows and no_empty:
2511+
if not rows:
25112512
logger.error(f"No data rows in {filename}!")
25122513
return False, retdict
25132514
if yearcol is not None or date_function is not None:

tests/hdx/data/test_dataset_resource_generation.py

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -500,6 +500,23 @@ def process_year(row):
500500
)
501501
assert success is False
502502
url = "https://raw.githubusercontent.com/OCHA-DAP/hdx-python-api/main/tests/fixtures/gen_resource/test_data_no_data.csv"
503+
success, results = dataset.download_generate_resource(
504+
downloader,
505+
url,
506+
folder,
507+
filename,
508+
resourcedata,
509+
)
510+
assert success is False
511+
success, results = dataset.download_generate_resource(
512+
downloader,
513+
url,
514+
folder,
515+
filename,
516+
resourcedata,
517+
no_empty=False,
518+
)
519+
assert success is True
503520
success, results = dataset.download_generate_resource(
504521
downloader,
505522
url,

0 commit comments

Comments
 (0)