Test table comment in Faker

[sthandassery]

Description

Adding test coverage for setTableProperties. As part of the unit test getTableComment utility is now moved down from BaseConnectorTest to AbstractTestQueryFramework class, so that TestFakerQueries can reuse it.

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.

[cla-bot]

Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to [email protected]. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla

[nineinchnick]

The commit message and PR title are wrong

[ebyhr]

Fixes #23911

Please don't use "Fixes" comment. We should add more tests in #23911.

[sthandassery]

The commit message and PR title are wrong

@nineinchnick Do you mean - moving the getTableComment utility needs to extracted out into its own commit?

[nineinchnick]

This PR only moves getTableComment, so there's nothing to extract - change the commit message and PR title/description to explain why it's being moved.

[github-actions]

This pull request has gone a while without any activity. Tagging for triage help: @mosabua

[mosabua]

Can you follow up on the review comments @sthandassery and then @nineinchnick and @raunaqmorarka can proceed towards merge.