PYTHON-4701 - Topology logging should use suppress_event

[NoahStapp]

No description provided.

[ShaneHarvey]

Should this be and not suppress_event?

[NoahStapp]

suppress_event checks for event publishing flags, which we don't use for logging.

[ShaneHarvey]

Could that be changed?

suppress_event = sd_old == server_description

[ShaneHarvey]

Also could you add some more info to the failing assert message?

[NoahStapp]

Also could you add some more info to the failing assert message?

To which message?

[ShaneHarvey]

The one in this ticket:

TestUnifiedLoggingStandalone.test_Topology_lifecycle _____________

self = <test.test_discovery_and_monitoring.TestUnifiedLoggingStandalone testMethod=test_Topology_lifecycle>

    def test_case(self):
>       self.run_scenario(spec)

test/unified_format.py:2074: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
test/unified_format.py:2018: in run_scenario
    self._run_scenario(spec, uri)
test/unified_format.py:2048: in _run_scenario
    self.check_log_messages(spec["operations"], expect_log_messages)
test/unified_format.py:1957: in check_log_messages
    self.assertEqual(len(client["messages"]), len(actual_logs))
E   AssertionError: 7 != 8

[ShaneHarvey]

Could you update the PR title and Jira ticket to describe the bug fix instead of the test failure name?