fix rich markup regression in 0.19.7

[noklam]

Description

fix #4094

The root cause is the previous test aren't testing the correct thing. The problem is RichHandler is in the root logger but not DataCatalog internal one.

Development notes

• Moved the test to test_session.py, as DataCatalog does not related to RichHandler. The RichHandler is more for Kedro project. This mean when someone use DataCatalog alone they won't see the markup log, but that's fine since they are not using RichHandler in that case.
• Remove lru_cache, the internal function is not used anywhere else and is not a user function. I change it to a internal attribute to DataCatalog.

Developer Certificate of Origin

We need all contributions to comply with the Developer Certificate of Origin (DCO). All commits must be signed off by including a Signed-off-by line in the commit message. See our wiki for guidance.

If your PR is blocked due to unsigned commits, then you must follow the instructions under "Rebase the branch" on the GitHub Checks page for your PR. This will retroactively add the sign-off to all unsigned commits and allow the DCO check to pass.

Checklist

• Read the contributing guidelines
• Signed off each commit with a Developer Certificate of Origin (DCO)
• Opened this PR as a 'Draft Pull Request' if it is work-in-progress
• Updated the documentation to reflect the code changes
• Added a description of this change in the RELEASE.md file
• Added tests to cover my changes
• Checked if this change will affect Kedro-Viz, and if so, communicated that with the Viz team

[lrcouto]

LGTM!

[merelcht]

I've tried this code and compared it to 0.19.7 and can confirm this now correctly shows highlighting again when using rich and also allows to run Kedro without rich installed! 👍