From 51825b18ba4f2e25de58b4a413ee26a6a8fdb268 Mon Sep 17 00:00:00 2001 From: Clebert Suconic Date: Thu, 18 Aug 2022 16:07:19 -0400 Subject: [PATCH] NO-JIRA: remove now-stale comments on tests that I missed when switching to cp files in 71b781d9193c64e34e9d986939af40745ea9740b --- .../AddressPauseJournalCompatibilityTest.java | 13 ------------- ...ectionFactoryConfigurationSerializationTest.java | 13 ------------- .../tests/compatibility/ExportImportTest.java | 13 ------------- .../tests/compatibility/HQLargeMeshTest.java | 13 ------------- .../artemis/tests/compatibility/HQSelectorTest.java | 13 ------------- .../tests/compatibility/HornetQSoakTest.java | 13 ------------- .../compatibility/JournalCompatibilityTest.java | 13 ------------- .../artemis/tests/compatibility/Mesh2Test.java | 13 ------------- .../artemis/tests/compatibility/MeshTest.java | 13 ------------- .../tests/compatibility/OldOpenWireTest.java | 13 ------------- .../QueuePauseJournalCompatibilityTest.java | 13 ------------- .../artemis/tests/compatibility/ReplyToTest.java | 13 ------------- .../artemis/tests/compatibility/SendAckTest.java | 13 ------------- .../tests/compatibility/SerializationTest.java | 13 ------------- .../artemis/tests/compatibility/package-info.java | 3 +-- 15 files changed, 1 insertion(+), 184 deletions(-) diff --git a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/AddressPauseJournalCompatibilityTest.java b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/AddressPauseJournalCompatibilityTest.java index a37e1bff2f..02ac664658 100644 --- a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/AddressPauseJournalCompatibilityTest.java +++ b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/AddressPauseJournalCompatibilityTest.java @@ -32,19 +32,6 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.junit.runners.Parameterized; -/** - * To run this test on the IDE and debug it, run the compatibility-tests through a command line once: - * - * cd /compatibility-tests - * mvn install -Ptests | tee output.log - * - * on the output.log you will see the output generated by {@link #getClasspath(String)} - * - * On your IDE, edit the Run Configuration to your test and add those -D as parameters to your test. - * On Idea you would do the following: - * - * Run->Edit Configuration->Add ArtemisMeshTest and add your properties. - */ @RunWith(Parameterized.class) public class AddressPauseJournalCompatibilityTest extends VersionedBase { diff --git a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/ConnectionFactoryConfigurationSerializationTest.java b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/ConnectionFactoryConfigurationSerializationTest.java index 18bcd7bfd7..ea6e6c7e52 100644 --- a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/ConnectionFactoryConfigurationSerializationTest.java +++ b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/ConnectionFactoryConfigurationSerializationTest.java @@ -34,19 +34,6 @@ import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.ONE_FIVE import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.SNAPSHOT; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.TWO_FOUR; -/** - * To run this test on the IDE and debug it, run the compatibility-tests through a command line once: - * - * cd /compatibility-tests - * mvn install -Ptests | tee output.log - * - * on the output.log you will see the output generated by {@link #getClasspath(String)} - * - * On your IDE, edit the Run Configuration to your test and add those -D as parameters to your test. - * On Idea you would do the following: - * - * Run->Edit Configuration->Add ArtemisMeshTest and add your properties. - */ @RunWith(Parameterized.class) public class ConnectionFactoryConfigurationSerializationTest extends VersionedBase { diff --git a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/ExportImportTest.java b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/ExportImportTest.java index f7852b40ce..e487ecf73f 100644 --- a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/ExportImportTest.java +++ b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/ExportImportTest.java @@ -33,19 +33,6 @@ import org.junit.runners.Parameterized; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.ONE_FIVE; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.SNAPSHOT; -/** - * To run this test on the IDE and debug it, run the compatibility-tests through a command line once: - * - * cd /compatibility-tests - * mvn install -Ptests | tee output.log - * - * on the output.log you will see the output generated by {@link #getClasspath(String)} - * - * On your IDE, edit the Run Configuration to your test and add those -D as parameters to your test. - * On Idea you would do the following: - * - * Run->Edit Configuration->Add ArtemisMeshTest and add your properties. - */ @RunWith(Parameterized.class) public class ExportImportTest extends VersionedBase { private String serverScriptToUse; diff --git a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/HQLargeMeshTest.java b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/HQLargeMeshTest.java index 0b65c50cb3..862d1d09a9 100644 --- a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/HQLargeMeshTest.java +++ b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/HQLargeMeshTest.java @@ -31,19 +31,6 @@ import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.HORNETQ_ import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.HORNETQ_247; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.SNAPSHOT; -/** - * To run this test on the IDE and debug it, run the compatibility-tests through a command line once: - * - * cd /compatibility-tests - * mvn install -Ptests | tee output.log - * - * on the output.log you will see the output generated by {@link #getClasspath(String)} - * - * On your IDE, edit the Run Configuration to your test and add those -D as parameters to your test. - * On Idea you would do the following: - * - * Run->Edit Configuration->Add ArtemisMeshTest and add your properties. - */ @RunWith(Parameterized.class) public class HQLargeMeshTest extends VersionedBase { diff --git a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/HQSelectorTest.java b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/HQSelectorTest.java index c12b04c298..1a006e66f7 100644 --- a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/HQSelectorTest.java +++ b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/HQSelectorTest.java @@ -31,19 +31,6 @@ import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.HORNETQ_ import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.HORNETQ_247; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.SNAPSHOT; -/** - * To run this test on the IDE and debug it, run the compatibility-tests through a command line once: - * - * cd /compatibility-tests - * mvn install -Ptests | tee output.log - * - * on the output.log you will see the output generated by {@link #getClasspath(String)} - * - * On your IDE, edit the Run Configuration to your test and add those -D as parameters to your test. - * On Idea you would do the following: - * - * Run->Edit Configuration->Add ArtemisMeshTest and add your properties. - */ @RunWith(Parameterized.class) public class HQSelectorTest extends VersionedBase { diff --git a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/HornetQSoakTest.java b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/HornetQSoakTest.java index 7467a43c54..41ab2ccf33 100644 --- a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/HornetQSoakTest.java +++ b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/HornetQSoakTest.java @@ -33,19 +33,6 @@ import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.HORNETQ_ import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.ONE_FIVE; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.SNAPSHOT; -/** - * To run this test on the IDE and debug it, run the compatibility-tests through a command line once: - * - * cd /compatibility-tests - * mvn install -Ptests | tee output.log - * - * on the output.log you will see the output generated by {@link #getClasspath(String)} - * - * On your IDE, edit the Run Configuration to your test and add those -D as parameters to your test. - * On Idea you would do the following: - * - * Run->Edit Configuration->Add ArtemisMeshTest and add your properties. - */ public class HornetQSoakTest extends ClasspathBase { ClassLoader artemisClassLoader; diff --git a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/JournalCompatibilityTest.java b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/JournalCompatibilityTest.java index 9f9e02d291..89c854b2e9 100644 --- a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/JournalCompatibilityTest.java +++ b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/JournalCompatibilityTest.java @@ -34,19 +34,6 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.junit.runners.Parameterized; -/** - * To run this test on the IDE and debug it, run the compatibility-tests through a command line once: - * - * cd /compatibility-tests - * mvn install -Ptests | tee output.log - * - * on the output.log you will see the output generated by {@link #getClasspath(String)} - * - * On your IDE, edit the Run Configuration to your test and add those -D as parameters to your test. - * On Idea you would do the following: - * - * Run->Edit Configuration->Add ArtemisMeshTest and add your properties. - */ @RunWith(Parameterized.class) public class JournalCompatibilityTest extends VersionedBase { diff --git a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/Mesh2Test.java b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/Mesh2Test.java index de10c41a8b..9cb1d797d8 100644 --- a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/Mesh2Test.java +++ b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/Mesh2Test.java @@ -33,19 +33,6 @@ import org.junit.runners.Parameterized; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.SNAPSHOT; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.TWO_FOUR; -/** - * To run this test on the IDE and debug it, run the compatibility-tests through a command line once: - * - * cd /compatibility-tests - * mvn install -Ptests | tee output.log - * - * on the output.log you will see the output generated by {@link #getClasspath(String)} - * - * On your IDE, edit the Run Configuration to your test and add those -D as parameters to your test. - * On Idea you would do the following: - * - * Run->Edit Configuration->Add ArtemisMeshTest and add your properties. - */ @RunWith(Parameterized.class) public class Mesh2Test extends ServerBase { diff --git a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/MeshTest.java b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/MeshTest.java index d37f8e5e9b..6f5bf4dca7 100644 --- a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/MeshTest.java +++ b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/MeshTest.java @@ -37,19 +37,6 @@ import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.ONE_FOUR import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.SNAPSHOT; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.TWO_FOUR; -/** - * To run this test on the IDE and debug it, run the compatibility-tests through a command line once: - * - * cd /compatibility-tests - * mvn install -Ptests | tee output.log - * - * on the output.log you will see the output generated by {@link #getClasspath(String)} - * - * On your IDE, edit the Run Configuration to your test and add those -D as parameters to your test. - * On Idea you would do the following: - * - * Run->Edit Configuration->Add ArtemisMeshTest and add your properties. - */ @RunWith(Parameterized.class) public class MeshTest extends ServerBase { diff --git a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/OldOpenWireTest.java b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/OldOpenWireTest.java index ed47b86bf4..de34eb1d31 100644 --- a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/OldOpenWireTest.java +++ b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/OldOpenWireTest.java @@ -37,19 +37,6 @@ import org.junit.Test; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.AMQ_5_11; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.SNAPSHOT; -/** - * To run this test on the IDE and debug it, run the compatibility-tests through a command line once: - * - * cd /compatibility-tests - * mvn install -Ptests | tee output.log - * - * on the output.log you will see the output generated by {@link #getClasspath(String)} - * - * On your IDE, edit the Run Configuration to your test and add those -D as parameters to your test. - * On Idea you would do the following: - * - * Run->Edit Configuration->Add ArtemisMeshTest and add your properties. - */ public class OldOpenWireTest extends ClasspathBase { EmbeddedActiveMQ server; diff --git a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/QueuePauseJournalCompatibilityTest.java b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/QueuePauseJournalCompatibilityTest.java index 3b7b34244e..34048fa1be 100644 --- a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/QueuePauseJournalCompatibilityTest.java +++ b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/QueuePauseJournalCompatibilityTest.java @@ -32,19 +32,6 @@ import org.junit.runners.Parameterized; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.SNAPSHOT; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.TWO_SIX_THREE; -/** - * To run this test on the IDE and debug it, run the compatibility-tests through a command line once: - * - * cd /compatibility-tests - * mvn install -Ptests | tee output.log - * - * on the output.log you will see the output generated by {@link #getClasspath(String)} - * - * On your IDE, edit the Run Configuration to your test and add those -D as parameters to your test. - * On Idea you would do the following: - * - * Run->Edit Configuration->Add ArtemisMeshTest and add your properties. - */ @RunWith(Parameterized.class) public class QueuePauseJournalCompatibilityTest extends VersionedBase { diff --git a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/ReplyToTest.java b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/ReplyToTest.java index e7dd934766..202263dfa3 100644 --- a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/ReplyToTest.java +++ b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/ReplyToTest.java @@ -37,19 +37,6 @@ import org.junit.runners.Parameterized; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.ONE_FIVE; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.SNAPSHOT; -/** - * To run this test on the IDE and debug it, run the compatibility-tests through a command line once: - * - * cd /compatibility-tests - * mvn install -Ptests | tee output.log - * - * on the output.log you will see the output generated by {@link #getClasspath(String)} - * - * On your IDE, edit the Run Configuration to your test and add those -D as parameters to your test. - * On Idea you would do the following: - * - * Run->Edit Configuration->Add ArtemisMeshTest and add your properties. - */ @RunWith(Parameterized.class) public class ReplyToTest extends ServerBase { diff --git a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/SendAckTest.java b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/SendAckTest.java index 9df38e339a..4f03069423 100644 --- a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/SendAckTest.java +++ b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/SendAckTest.java @@ -31,19 +31,6 @@ import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.SNAPSHOT import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.TWO_FOUR; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.ONE_FIVE; -/** - * To run this test on the IDE and debug it, run the compatibility-tests through a command line once: - * - * cd /compatibility-tests - * mvn install -Ptests | tee output.log - * - * on the output.log you will see the output generated by {@link #getClasspath(String)} - * - * On your IDE, edit the Run Configuration to your test and add those -D as parameters to your test. - * On Idea you would do the following: - * - * Run->Edit Configuration->Add ArtemisMeshTest and add your properties. - */ @RunWith(Parameterized.class) public class SendAckTest extends ServerBase { diff --git a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/SerializationTest.java b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/SerializationTest.java index 3bd61b6d6f..331adeb188 100644 --- a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/SerializationTest.java +++ b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/SerializationTest.java @@ -34,19 +34,6 @@ import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.ONE_FIVE import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.SNAPSHOT; import static org.apache.activemq.artemis.tests.compatibility.GroovyRun.TWO_TEN_ZERO; -/** - * To run this test on the IDE and debug it, run the compatibility-tests through a command line once: - * - * cd /compatibility-tests - * mvn install -Ptests | tee output.log - * - * on the output.log you will see the output generated by {@link #getClasspath(String)} - * - * On your IDE, edit the Run Configuration to your test and add those -D as parameters to your test. - * On Idea you would do the following: - * - * Run->Edit Configuration->Add ArtemisMeshTest and add your properties. - */ @RunWith(Parameterized.class) public class SerializationTest extends VersionedBase { diff --git a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/package-info.java b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/package-info.java index e78ed58299..2ac7ce7581 100644 --- a/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/package-info.java +++ b/tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/package-info.java @@ -20,8 +20,7 @@ * and within some specific hornetQ versions. * * The integration tests pom will use the artemis-maven-plugin/dependency-scan to lookup for specific verions. - * The tests will only work if the plugin was executed and the system property was filled, - * or if it was manually added within the IDE running for debug purposes. + * The tests will only work if the plugin was executed and the file with a definition to the class (.CP files) is found */ package org.apache.activemq.artemis.tests.compatibility; \ No newline at end of file