Skip to content

Rewrite BrokerServiceTest#testBrokerStatsTopicLoadFailed #20495

@lhotari

Description

@lhotari

The BrokerServiceTest#testBrokerStatsTopicLoadFailed is disabled by #20494.

  1. Remove the test case and if the coverage is necessary, file a new ticket to add it back.

Originally posted by @tisonkun in #20494 (review)

The test BrokerServiceTest#testBrokerStatsTopicLoadFailed introduced in #19236 is invalid.

  • It modifies the current shared state and that causes other tests to fail.
  • The test uses Awaitility with a 2 minute timeout. That is too long for unit tests.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions