Fix toggle column action in the discover page

[tungsheng]

Description

The DocViewerTab component needs to be updated when the columns prop is updated.

This PR

• Add the columns prop check in the shouldComponentUpdate lifecycle method.
• Add unit test to test if component update when columns prop changed.

Issues Resolved

fixes

• [BUG] Toggle column in document doesn't toggle, it adds another column #8698

Screenshot

Testing the changes

Changelog

• fix: Fix toggle column action in the discover page

Check List

• All tests pass
  • yarn test:jest
  • yarn test:jest_integration
• New functionality includes testing.
• New functionality has been documented.
• Update CHANGELOG.md
• Commits are signed per the DCO using --signoff

[github-actions]

ℹ️ Manual Changeset Creation Reminder

Please ensure manual commit for changeset file 8905.yml under folder changelogs/fragments to complete this PR.

If you want to use the available OpenSearch Changeset Bot App to avoid manual creation of changeset file you can install it in your forked repository following this link.

For more information about formatting of changeset files, please visit OpenSearch Auto Changeset and Release Notes Tool.

[tungsheng]

@abbyhu2000 Would you kindly help with the WhiteSource Security Check fail in this PR? Any required action on my side? Is there a way to re-run the checks? Thank you! 🙏

[github-actions]

❌ Empty Changelog Section

The Changelog section in your PR description is empty. Please add a valid changelog entry or entries. If you did add a changelog entry, check to make sure that it was not accidentally included inside the comment block in the Changelog section.

[abbyhu2000]

@abbyhu2000 Would you kindly help with the WhiteSource Security Check fail in this PR? Any required action on my side? Is there a way to re-run the checks? Thank you! 🙏

let me re-run the check and see if it passes~

[abbyhu2000]

nice! Thanks for the unit test

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.84%. Comparing base (9f23442) to head (39f23f7).
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8905      +/-   ##
==========================================
- Coverage   60.87%   60.84%   -0.04%     
==========================================
  Files        3808     3808              
  Lines       91209    91209              
  Branches    14410    14410              
==========================================
- Hits        55526    55492      -34     
- Misses      32142    32222      +80     
+ Partials     3541     3495      -46     

Flag	Coverage Δ
Linux_1	29.02% <ø> (ø)
Linux_2	?
Linux_3	37.93% <ø> (+<0.01%)	⬆️
Linux_4	29.01% <ø> (ø)
Windows_1	29.03% <ø> (-0.02%)	⬇️
Windows_2	56.34% <ø> (ø)
Windows_3	37.93% <ø> (ø)
Windows_4	29.01% <ø> (ø)

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

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

[tungsheng]

@abbyhu2000 I am not sure how to trigger the WhiteSource Security Check from my side. Could you help review and trigger the check? or let me know what I can do to trigger, thank you for your help!

[tungsheng]

@abbyhu2000 @sejli Would you kindly help look after this PR? Please do let me know if there's more changes/actions needed to get this PR merged. Thank you very much for your help! 🙏

[tungsheng]

@abbyhu2000 @sejli @AMoo-Miki Just want to follow up with the PR review request. Please let me know the general process of working with the maintainer on merging a PR. I really appreciate your support!!

[sulabhjain-dev]

Thanks @tungsheng for moving this PR forward for review with OpenSearch Dashboards team.

We will also like to have it part of next OpenSearch Dashboards release as it is impacting critical customer workflow. It will be good to have it back-ported to 2.x branch.