fix(plugin-chart-handlebars): Fix TypeError when using handlebars columns raw mode

[fmannhardt]

SUMMARY

As already indicated in issue #22548 there is a TypeError when using the Handlebars visualization in the raw record mode. This seems to have been caused by adding the columns as an object instead of the expected array. I have simply taken the relevant code from the table plugin and replaced the invalid code with it.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

From #22548:

Local testing after the fix:

TESTING INSTRUCTIONS

• Load any dataset into a Handlebars chart
• Switch to raw records query mode
• Verify that there is no TypeError

ADDITIONAL INFORMATION

• Has associated issue: Handlebar multiple problems #22548
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[github-actions]

Congrats on making your first PR and thank you for contributing to Superset! 🎉 ❤️

We hope to see you in our Slack community too! Not signed up? Use our Slack App to self-register.

[rusackas]

Thanks for this! Running CI and pinging some additional reviewers.

[rusackas]

This PR seems to be stuck on a failing unit test. @fmannhardt I suspect a rebase from master might fix this.

[rusackas]

@fmannhardt This seems to be failing repeatedly on unit tests. Could be flaky tests, but I haven't investigated. You might get lucky with a rebase from master fixing it (I think there was a problematic one that was fixed) but otherwise you might need to run and debug the tests a bit locally.

[fmannhardt]

I have rebased on current master and pushed the changes.

[codecov]

Codecov Report

Merging #23801 (c29aaca) into master (7a55625) will increase coverage by 0.00%.
The diff coverage is 0.00%.

❗ Current head c29aaca differs from pull request most recent head fc6d026. Consider uploading reports for the commit fc6d026 to get more accurate results

@@           Coverage Diff           @@
##           master   #23801   +/-   ##
=======================================
  Coverage   66.54%   66.54%           
=======================================
  Files        1941     1941           
  Lines       75280    75279    -1     
  Branches     8168     8168           
=======================================
  Hits        50093    50093           
+ Misses      23098    23097    -1     
  Partials     2089     2089           

Flag	Coverage Δ
javascript	54.52% <0.00%> (+<0.01%)	⬆️

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

Impacted Files	Coverage Δ
...n-chart-handlebars/src/plugin/controls/columns.tsx	15.78% <0.00%> (+0.78%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[fmannhardt]

@rusackas, the rebase seems to have done the trick.