You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by lh...@apache.org on 2022/06/05 11:22:44 UTC
[pulsar] branch master updated: Fix flaky test PersistentTopicsTest#testTriggerCompactionTopic. (#15937)
This is an automated email from the ASF dual-hosted git repository.
lhotari pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 91cb00eab0f Fix flaky test PersistentTopicsTest#testTriggerCompactionTopic. (#15937)
91cb00eab0f is described below
commit 91cb00eab0fbbc0d519f5ad1ac8ab722da56b349
Author: Jiwei Guo <te...@apache.org>
AuthorDate: Sun Jun 5 19:22:36 2022 +0800
Fix flaky test PersistentTopicsTest#testTriggerCompactionTopic. (#15937)
---
.../java/org/apache/pulsar/broker/admin/PersistentTopicsTest.java | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/PersistentTopicsTest.java b/pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/PersistentTopicsTest.java
index 8155ffadbb7..356367f410a 100644
--- a/pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/PersistentTopicsTest.java
+++ b/pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/PersistentTopicsTest.java
@@ -901,9 +901,13 @@ public class PersistentTopicsTest extends MockedPulsarServiceBaseTest {
// create non partitioned topic and compaction on it
response = mock(AsyncResponse.class);
+ ArgumentCaptor<Response> responseCaptor = ArgumentCaptor.forClass(Response.class);
persistentTopics.createNonPartitionedTopic(response, testTenant, testNamespace, nonPartitionTopicName, true, null);
+ verify(response, timeout(5000).times(1)).resume(responseCaptor.capture());
+ Assert.assertEquals(responseCaptor.getValue().getStatus(), Response.Status.NO_CONTENT.getStatusCode());
+ response = mock(AsyncResponse.class);
+ responseCaptor = ArgumentCaptor.forClass(Response.class);
persistentTopics.compact(response, testTenant, testNamespace, nonPartitionTopicName, true);
- ArgumentCaptor<Response> responseCaptor = ArgumentCaptor.forClass(Response.class);
verify(response, timeout(5000).times(1)).resume(responseCaptor.capture());
Assert.assertEquals(responseCaptor.getValue().getStatus(), Response.Status.NO_CONTENT.getStatusCode());