Fix: Skip type comparison if disableTypeComments is true

[berkut1]

Q	A
Type	bug
Fixed issues	#6422

Summary

In the #6150, a check was missed when we were validating the database schema.

[greg0ire]

Can you please add a unit test?

[berkut1]

How do I test this? I tried to understand it by looking at the previous pull request #6150, but I didn't find any tests there. I guess I need to create a Comparator object with a Platform, but in AbstractComparatorTestCase Class, no object is created in any test except setUp().
Do I really need to create it in my test function and pass disableTypeComments? It seems strange to me considering all the other tests.

Or should I write a test in AbstractPlatformTestCase?

[berkut1]

Do you think such a test would be suitable? After reviewing AbstractPlatformTestCase, it seems like the most appropriate place for it.

[greg0ire]

The place seems appropriate. I don't think you need to create a Comparator though.

Didn't mean to approve yet, only comment for now, but after the suggested change it should be good

[berkut1]

Thanks, I'm fine with those changes.

[SenseException]

The namings are weird to read without the context of the issues #6422 and #6257 😅

[greg0ire]

Thanks @berkut1 !