fix(python): Ensure read_excel and read_ods return identical frames across all engines when given empty spreadsheet tables

[alexander-beedie]

Ref: #16408 (though doesn't close it - related, but not the same).

Extended test coverage to handle both the "no data" and "no rows" cases to ensure that we return identical frames in all cases (eg: if we have column names but no table rows we assign String dtype to empty frame cols).

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.38%. Comparing base (38149d6) to head (20fda4e).

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #16802      +/-   ##
==========================================
- Coverage   81.39%   81.38%   -0.02%     
==========================================
  Files        1424     1424              
  Lines      187623   187625       +2     
  Branches     2691     2692       +1     
==========================================
- Hits       152719   152690      -29     
- Misses      34409    34440      +31     
  Partials      495      495              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ritchie46]

Can we write ods files? Otherwise I'd rather have that we write them on session instead of adding to the repo.

[alexander-beedie]

Can we write ods files? Otherwise I'd rather have that we write them on session instead of adding to the repo.

While we can now read ods, xls, xlsb and xlsx (and -I think- xlsm), we can only write to xlsx.

[ritchie46]

Alright, repo it is then. 👍