Skip to content

[cleanup] Convert 13 test classes to SharedPulsarBaseTest#25331

Open
merlimat wants to merge 5 commits intoapache:masterfrom
merlimat:shared-tests-convert-13a
Open

[cleanup] Convert 13 test classes to SharedPulsarBaseTest#25331
merlimat wants to merge 5 commits intoapache:masterfrom
merlimat:shared-tests-convert-13a

Conversation

@merlimat
Copy link
Contributor

Convert 13 test classes from ProducerConsumerBase to SharedPulsarBaseTest, reducing
test execution time by sharing a single Pulsar cluster across tests.

Converted classes:

  • CompactedOutBatchMessageTest
  • ConsumeBaseExceptionTest
  • ConsumerCloseTest
  • ConsumerDedupPermitsUpdateTest
  • ConsumerMemoryLimitTest
  • ConsumerUnsubscribeIntegrationTest
  • DispatchAccordingPermitsTest
  • ExposeMessageRedeliveryCountTest
  • HierarchyTopicAutoCreationTest
  • MemoryLimitTest
  • ProduceWithMessageIdTest
  • SimpleTypedProducerConsumerTest
  • TopicFromMessageTest

Documentation

  • doc-not-needed (Changes in your PR don't need doc changes)

Matching PR in forked repository

PR in forked repository:

BytesKeyTest, ConsumerAckListTest, ConsumerCleanupTest, CustomMessageIdTest,
FailoverSubscriptionTest, NonPartitionedTopicExpectedTest,
PersistentTopicTerminateTest, ProducerCleanupTest, ProducerQueueSizeTest,
ReplicateSubscriptionTest
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Mar 17, 2026
@merlimat merlimat force-pushed the shared-tests-convert-13a branch 3 times, most recently from 3d53567 to 81f3853 Compare March 17, 2026 01:32
CompactedOutBatchMessageTest, ConsumeBaseExceptionTest, ConsumerCloseTest,
ConsumerDedupPermitsUpdateTest, ConsumerMemoryLimitTest,
ConsumerUnsubscribeIntegrationTest, DispatchAccordingPermitsTest,
ExposeMessageRedeliveryCountTest, HierarchyTopicAutoCreationTest,
MemoryLimitTest, ProduceWithMessageIdTest, SimpleTypedProducerConsumerTest,
TopicFromMessageTest
@merlimat merlimat force-pushed the shared-tests-convert-13a branch from 81f3853 to d564902 Compare March 17, 2026 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/test doc-not-needed Your PR changes do not impact docs ready-to-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants