diff options
| author | Robert Gemmell <robbie@apache.org> | 2010-09-08 10:07:19 +0000 |
|---|---|---|
| committer | Robert Gemmell <robbie@apache.org> | 2010-09-08 10:07:19 +0000 |
| commit | f83c155657ebd11272ad890df9d42c096e1ad50b (patch) | |
| tree | 8ecd3ec01b0b19b573e60a53755644c08b5e108a /qpid/java/systests | |
| parent | d48c1c5847c308e85a48813fc1c4e5904422606a (diff) | |
| download | qpid-python-f83c155657ebd11272ad890df9d42c096e1ad50b.tar.gz | |
QPID-2838, QPID-2843: test readability tweaks
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@994963 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'qpid/java/systests')
2 files changed, 5 insertions, 5 deletions
diff --git a/qpid/java/systests/src/main/java/org/apache/qpid/server/logging/AbstractTestLogging.java b/qpid/java/systests/src/main/java/org/apache/qpid/server/logging/AbstractTestLogging.java index 96cf2d9b70..9c1d302be0 100644 --- a/qpid/java/systests/src/main/java/org/apache/qpid/server/logging/AbstractTestLogging.java +++ b/qpid/java/systests/src/main/java/org/apache/qpid/server/logging/AbstractTestLogging.java @@ -465,7 +465,7 @@ public class AbstractTestLogging extends QpidBrokerTestCase System.err.println(log); } - if (results.size() == 0 && monitor != null) + if (results.isEmpty() && monitor != null) { System.err.println("Monitored file contents:"); System.err.println(monitor.readFile()); diff --git a/qpid/java/systests/src/main/java/org/apache/qpid/server/logging/ManagementLoggingTest.java b/qpid/java/systests/src/main/java/org/apache/qpid/server/logging/ManagementLoggingTest.java index 6868f82ed9..7ab937b15f 100644 --- a/qpid/java/systests/src/main/java/org/apache/qpid/server/logging/ManagementLoggingTest.java +++ b/qpid/java/systests/src/main/java/org/apache/qpid/server/logging/ManagementLoggingTest.java @@ -219,10 +219,10 @@ public class ManagementLoggingTest extends AbstractTestLogging validateMessageID("MNG-1002", log); // We expect the RMI Registry port (the defined 'management port') to be - // 100 lower than the RMIConnector Server Port (the actual JMX server) - mPort = getPort() + (DEFAULT_MANAGEMENT_PORT - DEFAULT_PORT) + 100; - assertTrue("RMI ConnectorServer port not as expected(" + mPort + ").:" + getMessageString(log), - getMessageString(log).endsWith(String.valueOf(mPort))); + // 100 lower than the JMX RMIConnector Server Port (the actual JMX server) + int jmxPort = mPort + 100; + assertTrue("JMX RMIConnectorServer port not as expected(" + jmxPort + ").:" + getMessageString(log), + getMessageString(log).endsWith(String.valueOf(jmxPort))); } catch (AssertionFailedError afe) { |