fix: Handling no columns

[bmingles]

Added graceful handling to IrisGrid for tables with no columns.

fixes #1169

[codecov]

Codecov Report

Merging #1170 (740ea9b) into main (e314be6) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1170   +/-   ##
=======================================
  Coverage   44.15%   44.15%           
=======================================
  Files         447      447           
  Lines       33266    33267    +1     
  Branches     8355     8356    +1     
=======================================
+ Hits        14687    14688    +1     
  Misses      18530    18530           
  Partials       49       49           

Flag	Coverage Δ
unit	44.15% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
packages/iris-grid/src/IrisGrid.tsx	27.05% <100.00%> (+0.04%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[dsmmcken]

Are there other places we need to check for columns[0] being used in unsupported ways within iris grid?

[bmingles]

Are there other places we need to check for columns[0] being used in unsupported ways within iris grid?

@dsmmcken In the IrisGrid component I only see 1 other reference to partitionTable.columns[0]. The block containing this code is already in a try / catch.

I do see other iris-grid related components that have columns[0] references. It may be worth looking into these, but I think it will expand the scope + testing overhead of this issue.

[dsmmcken]

That's fine. Should probably make a follow-up ticket and audit iris grid to see if there are behaviours expecting columns to not be zero where we might also fail.

[bmingles]

I've created #1172 to investigate whether there are additional issues.