Fix for issue 6 (#9)

* Added code and test to account for empty cells 'None' > ''

* Added code and test to account for empty cells 'None' > ''
Fixed paths, removed print statement

Co-authored-by: BYB <privateemail>

Author: bai-yi-bai

sheet2dict/main.py

@@ -46,10 +46,10 @@ def _parse_sheet_items(self, sheet: Worksheet, parsing_all_sheets: bool = False)
         sheet_title = sheet.title
 
         def item(row, col):
-            return (
-                sheet.cell(row=1, column=col).value,
-                str(sheet.cell(row=row, column=col).value),
-            )
+            if sheet.cell(row=row, column=col).value is None:
+                return (sheet.cell(row=1, column=col).value, '')
+            else:
+                return (sheet.cell(row=1, column=col).value, str(sheet.cell(row=row, column=col).value))
 
         list_to_append = (
             self.sheet_items[sheet_title] if parsing_all_sheets else self.sheet_items

tests/test_3_xlsx.py

@@ -32,6 +32,7 @@ def test_parse_xlsx_sheet_items(worksheet):
     assert "Taipei" in str(ws_items)
     assert "Bratislava" not in str(ws_items)
     assert "None:" in str(ws_items)
+    assert "'None'," not in str(ws_items)  # Tests whether empty cells were converted to "" strings and not "None"
     assert None in ws_items[0]
     assert len(ws_items) > 1
     assert len(ws_items) == 6