Rich markup regression in 0.19.7

[noklam]

Context

The regression happened in 0.19.7.

before 0.19.7:

In 0.19.7:

However, I think the change introduced in #3682 never really worked properly. The logic to detect the rich handler doesn't seem to work. When I have rich installed the orange colouring isn't there, and I added some print statements and it seems like the RichHandler isn't ever set.

Originally posted by @merelcht in #4090 (comment)

[lrcouto]

I've found that the logger object being used in testing has nothing in its handlers parameter, this is why it's not working.

{'filters': [], 'name': 'kedro.io.data_catalog', 'level': 0, 'parent': <Logger kedro (INFO)>, 'propagate': True, 'handlers': [], 'disabled': False, '_cache': {20: True}, 'manager': <logging.Manager object at 0x7fea88d9d010>}

[noklam]

@lrcouto See #4097

The original tests are not that useful because they are not testing the actual logger (root) that do the work. I removed the cache as I don't think it's useful and simply move it as an internal attribute (self._use_rich_markup).

The original function try to be generic but I think it is a bit over-complicated for things that no one is using.

[lrcouto]

That seems like it works nicely! Works without Rich as expected too. I fixed that one linter error that was popping up too.