Apache
/
activemq-artemis
mirror of https://github.com/apache/activemq-artemis.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

NO-JIRA Simplifying Test 

No need to test functionality of the mirror. the intention was if the messages would arrive correctly.

Other tests are verfying functionality.
This commit is contained in:
Clebert Suconic 2024-06-01 14:16:38 -04:00
parent ae92b714e3
commit b9bb494b58
1 changed files with 13 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/MirroredVersionTest.java
View File

@ -32,13 +32,10 @@ import java.util.ArrayList;
import java.util.Collection;
import java.util.List;
import org.apache.activemq.artemis.api.core.management.SimpleManagement;
import org.apache.activemq.artemis.tests.compatibility.base.ClasspathBase;
import org.apache.activemq.artemis.utils.FileUtil;
import org.apache.activemq.artemis.utils.Wait;
import org.apache.qpid.jms.JmsConnectionFactory;
import org.junit.After;
import org.junit.Assert;
import org.junit.Before;
import org.junit.Test;
import org.junit.runner.RunWith;
@ -152,9 +149,7 @@ public class MirroredVersionTest extends ClasspathBase {
connection.start();
for (int i = 0; i < 10; i++) {
TextMessage message = (TextMessage) consumer.receive(5000);
Assert.assertNotNull(message);
Assert.assertEquals("hello " + i + body, message.getText());
Assert.assertEquals(i, message.getIntProperty("count"));
if (message == null) break;
}
session.rollback();
}
@ -170,9 +165,9 @@ public class MirroredVersionTest extends ClasspathBase {
connection.start();
for (int i = 0; i < 10; i++) {
TextMessage message = (TextMessage) consumer.receive(5000);
Assert.assertNotNull(message);
Assert.assertEquals("hello " + i + body, message.getText());
Assert.assertEquals(i, message.getIntProperty("count"));
if (message == null) {
break;
}
}
session.commit();
}
@ -244,7 +239,9 @@ public class MirroredVersionTest extends ClasspathBase {
MessageConsumer consumer = session.createDurableConsumer(topic, sub1);
for (int i = 0; i < 10; i++) {
TextMessage message = (TextMessage)consumer.receive(5000);
Assert.assertNotNull(message);
if (message == null) {
break;
}
}
session.rollback();
}
@ -261,7 +258,9 @@ public class MirroredVersionTest extends ClasspathBase {
MessageConsumer consumer = session.createDurableConsumer(topic, sub1);
for (int i = 0; i < 10; i++) {
TextMessage message = (TextMessage)consumer.receive(5000);
Assert.assertNotNull(message);
if (message == null) {
break;
}
}
session.commit();
}
@ -274,16 +273,12 @@ public class MirroredVersionTest extends ClasspathBase {
MessageConsumer consumer = session.createDurableConsumer(topic, sub2);
for (int i = 0; i < 10; i++) {
TextMessage message = (TextMessage)consumer.receive(5000);
Assert.assertNotNull(message);
if (message == null) {
break;
}
}
session.commit();
}
if (useDual) {
SimpleManagement simpleManagementMainServer = new SimpleManagement("tcp://localhost:61616", null, null);
Wait.assertEquals(0, () -> simpleManagementMainServer.getMessageCountOnQueue(clientID1 + "." + sub1), 5000);
Wait.assertEquals(0, () -> simpleManagementMainServer.getMessageCountOnQueue(clientID2 + "." + sub2), 5000);
}
}
}