You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by or...@apache.org on 2019/07/26 15:27:02 UTC
[qpid-broker-j] 06/07: QPID-8350: Move getTestName() to
BrokerAdminUsingTestBase
This is an automated email from the ASF dual-hosted git repository.
orudyy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/qpid-broker-j.git
commit edc563d403dce72e22d50528c598c94ec351835b
Author: Alex Rudyy <or...@apache.org>
AuthorDate: Wed Jul 24 09:40:19 2019 +0100
QPID-8350: Move getTestName() to BrokerAdminUsingTestBase
---
.../java/org/apache/qpid/tests/http/HttpTestBase.java | 16 +++-------------
.../main/java/org/apache/qpid/systests/JmsTestBase.java | 12 +-----------
.../qpid/tests/utils/BrokerAdminUsingTestBase.java | 9 +++++++++
3 files changed, 13 insertions(+), 24 deletions(-)
diff --git a/systests/qpid-systests-http-management/src/main/java/org/apache/qpid/tests/http/HttpTestBase.java b/systests/qpid-systests-http-management/src/main/java/org/apache/qpid/tests/http/HttpTestBase.java
index 04f37ea..f1ecd4c 100644
--- a/systests/qpid-systests-http-management/src/main/java/org/apache/qpid/tests/http/HttpTestBase.java
+++ b/systests/qpid-systests-http-management/src/main/java/org/apache/qpid/tests/http/HttpTestBase.java
@@ -30,8 +30,6 @@ import javax.naming.NamingException;
import org.junit.After;
import org.junit.Before;
-import org.junit.Rule;
-import org.junit.rules.TestName;
import org.apache.qpid.server.model.Protocol;
import org.apache.qpid.systests.ConnectionBuilder;
@@ -44,9 +42,6 @@ public abstract class HttpTestBase extends BrokerAdminUsingTestBase
{
public static final String DEFAULT_BROKER_CONFIG = "classpath:config-http-management-tests.json";
- @Rule
- public final TestName _testName = new TestName();
-
private HttpTestHelper _helper;
private JmsProvider _jmsProvider;
@@ -72,12 +67,12 @@ public abstract class HttpTestBase extends BrokerAdminUsingTestBase
protected String getVirtualHost()
{
- return getClass().getSimpleName() + "_" + _testName.getMethodName();
+ return getClass().getSimpleName() + "_" + getTestName();
}
protected String getVirtualHostNode()
{
- return getClass().getSimpleName() + "_" + _testName.getMethodName();
+ return getClass().getSimpleName() + "_" + getTestName();
}
public HttpTestHelper getHelper()
@@ -102,7 +97,7 @@ public abstract class HttpTestBase extends BrokerAdminUsingTestBase
private HttpRequestConfig getHttpRequestConfig() throws Exception
{
- HttpRequestConfig config = getClass().getMethod(_testName.getMethodName(), new Class[]{}).getAnnotation(HttpRequestConfig.class);
+ HttpRequestConfig config = getClass().getMethod(getTestName(), new Class[]{}).getAnnotation(HttpRequestConfig.class);
if (config == null)
{
config = getClass().getAnnotation(HttpRequestConfig.class);
@@ -121,9 +116,4 @@ public abstract class HttpTestBase extends BrokerAdminUsingTestBase
return Utils.getProtocol();
}
- protected String getTestName()
- {
- return _testName.getMethodName();
- }
-
}
diff --git a/systests/qpid-systests-jms-core/src/main/java/org/apache/qpid/systests/JmsTestBase.java b/systests/qpid-systests-jms-core/src/main/java/org/apache/qpid/systests/JmsTestBase.java
index c264b6b..2c77f58 100644
--- a/systests/qpid-systests-jms-core/src/main/java/org/apache/qpid/systests/JmsTestBase.java
+++ b/systests/qpid-systests-jms-core/src/main/java/org/apache/qpid/systests/JmsTestBase.java
@@ -41,8 +41,6 @@ import javax.jms.TopicConnection;
import javax.naming.NamingException;
import org.junit.BeforeClass;
-import org.junit.Rule;
-import org.junit.rules.TestName;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
@@ -58,9 +56,6 @@ public abstract class JmsTestBase extends BrokerAdminUsingTestBase
private static JmsProvider _jmsProvider;
private static AmqpManagementFacade _managementFacade;
- @Rule
- public final TestName _testName = new TestName();
-
@BeforeClass
public static void setUpTestBase()
{
@@ -128,12 +123,7 @@ public abstract class JmsTestBase extends BrokerAdminUsingTestBase
protected String getVirtualHostName()
{
- return getClass().getSimpleName() + "_" + _testName.getMethodName();
- }
-
- protected String getTestName()
- {
- return _testName.getMethodName();
+ return getClass().getSimpleName() + "_" + getTestName();
}
protected Queue getQueue(String queueName) throws Exception
diff --git a/systests/systests-utils/src/main/java/org/apache/qpid/tests/utils/BrokerAdminUsingTestBase.java b/systests/systests-utils/src/main/java/org/apache/qpid/tests/utils/BrokerAdminUsingTestBase.java
index 73149af..852b732 100644
--- a/systests/systests-utils/src/main/java/org/apache/qpid/tests/utils/BrokerAdminUsingTestBase.java
+++ b/systests/systests-utils/src/main/java/org/apache/qpid/tests/utils/BrokerAdminUsingTestBase.java
@@ -20,6 +20,8 @@
package org.apache.qpid.tests.utils;
+import org.junit.Rule;
+import org.junit.rules.TestName;
import org.junit.runner.RunWith;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
@@ -28,6 +30,8 @@ import org.slf4j.LoggerFactory;
public abstract class BrokerAdminUsingTestBase
{
private static final Logger LOGGER = LoggerFactory.getLogger(BrokerAdminUsingTestBase.class);
+ @Rule
+ public final TestName _testName = new TestName();
private BrokerAdmin _brokerAdmin;
@@ -40,4 +44,9 @@ public abstract class BrokerAdminUsingTestBase
{
return _brokerAdmin;
}
+
+ protected String getTestName()
+ {
+ return _testName.getMethodName();
+ }
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org