[ntuple] Change GetColumn methods in Inspector

[magnustymoteus]

This Pull request:

Changes or fixes:

• Renamed GetColumnsByFieldId to GetAllColumnsOfField
• Added unit tests for GetAllColumnsOfField because it also recently became a public method

Checklist:

• tested changes locally
• updated the docs (if necessary)

[silverweed]

Not sure if we actually need the non-recursive GetColumnIds, considering that it's basically just a trivial filter on GetColumnIterable()...do you have a concrete use case for it?

[magnustymoteus]

I am currently using it in my treemap visualization, but yes one can easily replace it with a for loop over the GetColumnIterable. I'll remove the method then?

[magnustymoteus]

Alright, I've removed GetColumnIds(fieldId), but not GetColumnIds(colType) because it's used in GetPageSizeDistribution

[github-actions]

Test Results

    21 files      21 suites   3d 6h 52m 29s ⏱️
 3 223 tests  3 223 ✅ 0 💤 0 ❌
65 960 runs  65 960 ✅ 0 💤 0 ❌

Results for commit 6cb41c1.

♻️ This comment has been updated with latest results.