Skip to content

Commit

Permalink
fix: fix the grpc timeout of delay, order and simple test
Browse files Browse the repository at this point in the history
Signed-off-by: wangxye <[email protected]>
  • Loading branch information
wangxye committed Nov 14, 2023
1 parent 24d030f commit 321a411
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ public void testDelayTime15SecondsAgo() {
producer = ProducerFactory.getRMQProducer(account, topic);
Assertions.assertNotNull(producer, "Get Producer failed");
for (int i = 0; i < SEND_NUM; i++) {
Message message = MessageFactory.buildDelayMessage(topic, tag, RandomUtils.getStringByUUID(), System.currentTimeMillis());
Message message = MessageFactory.buildDelayMessage(topic, tag, RandomUtils.getStringByUUID(), System.currentTimeMillis() + 5 * 1000L);
producer.send(message);
}
Assertions.assertEquals(SEND_NUM, producer.getEnqueueMessages().getDataSize(), "send message failed");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@
import org.apache.rocketmq.listener.rmq.RMQNormalListener;
import org.apache.rocketmq.util.NameUtils;
import org.apache.rocketmq.util.RandomUtils;
import org.apache.rocketmq.util.TestUtils;
import org.apache.rocketmq.util.VerifyUtils;
import org.apache.rocketmq.util.data.collect.DataCollector;
import org.junit.jupiter.api.AfterEach;
Expand Down Expand Up @@ -99,6 +100,7 @@ public void testOrder_Send_PushConsumeOrderly() {
Assertions.assertEquals(SEND_NUM, producer.getEnqueueMessages().getDataSize(), "send message failed");
// DataCollector<Object> dequeueMessages = simpleConsumer.getListener().getDequeueMessages();
// dequeueMessages.addData(pushConsumer.getListener().getDequeueMessages());
// TestUtils.waitForSeconds(5);
VerifyUtils.verifyOrderMessage(producer.getEnqueueMessages(), pushConsumer.getListener().getDequeueMessages());
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ public void testFIFO_simple_receive_ack() {
String groupId = getOrderlyGroupId(methodName, SubscriptionMode.SUB_MODE_POP);

SimpleConsumer consumer = ConsumerFactory.getSimpleConsumer(account, topic, groupId, new FilterExpression(tag), Duration.ofSeconds(5));
VerifyUtils.tryReceiveOnce(consumer);
// VerifyUtils.tryReceiveOnce(consumer);
RMQNormalProducer producer = ProducerFactory.getRMQProducer(account, topic);
Assertions.assertNotNull(producer, "Get Producer failed");

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ public void test_waitAckException_reReceive_ack() {
if (messageViews.size() > 0) {
for (MessageView messageView : messageViews) {
log.info("MessageId:{}, Body:{}, tag:{}, property:{}", messageView.getMessageId(), StandardCharsets.UTF_8.decode(messageView.getBody()).toString(), messageView.getTag().get(), messageView.getProperties());
TestUtils.waitForSeconds(20);
TestUtils.waitForSeconds(10);
consumer.ack(messageView);
Assertions.fail("Calling changeInvisibleDuration after ack fails with an INVALID_RECEIPT_HANDLE error");
}
Expand Down

0 comments on commit 321a411

Please sign in to comment.