QPID-3610: set TCP_NODELAY to true by default, add new system property for changing default, add unit tests for system properties + connection url options.

git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@1200803 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 49 insertions, 0 deletions

diff --git a/qpid/java/common/src/test/java/org/apache/qpid/transport/ConnectionSettingsTest.java b/qpid/java/common/src/test/java/org/apache/qpid/transport/ConnectionSettingsTest.java
new file mode 100644
index 0000000000..62ca0b70ba
--- /dev/null
+++ b/qpid/java/common/src/test/java/org/apache/qpid/transport/ConnectionSettingsTest.java
@@ -0,0 +1,49 @@
+package org.apache.qpid.transport;
+
+import org.apache.qpid.configuration.ClientProperties;
+import org.apache.qpid.test.utils.QpidTestCase;
+
+public class ConnectionSettingsTest extends QpidTestCase
+{
+    ConnectionSettings _conConnectionSettings;
+
+    protected void setUp() throws Exception
+    {
+        super.setUp();
+        _conConnectionSettings = new ConnectionSettings();
+    }
+
+    public void testDefaultTCP_NODELAY()
+    {
+        assertTrue("Default for isTcpNodelay() should be true", _conConnectionSettings.isTcpNodelay());
+    }
+
+    public void testSystemPropertyOverrideTrueForTCP_NODELAY()
+    {
+        systemPropertyOverrideForTCP_NODELAYImpl(ClientProperties.QPID_TCP_NODELAY_PROP_NAME, true);
+    }
+    
+    public void testSystemPropertyOverrideFalseForTCP_NODELAY()
+    {
+        systemPropertyOverrideForTCP_NODELAYImpl(ClientProperties.QPID_TCP_NODELAY_PROP_NAME, false);
+    }
+
+    public void testLegacySystemPropertyOverrideTrueForTCP_NODELAY()
+    {
+        systemPropertyOverrideForTCP_NODELAYImpl(ClientProperties.AMQJ_TCP_NODELAY_PROP_NAME, true);
+    }
+
+    public void testLegacySystemPropertyOverrideFalseForTCP_NODELAY()
+    {
+        systemPropertyOverrideForTCP_NODELAYImpl(ClientProperties.AMQJ_TCP_NODELAY_PROP_NAME, false);
+    }
+
+    private void systemPropertyOverrideForTCP_NODELAYImpl(String propertyName, boolean value)
+    {
+        //set the default via system property
+        setTestSystemProperty(propertyName, String.valueOf(value));
+
+        _conConnectionSettings = new ConnectionSettings();
+        assertEquals("Value for isTcpNodelay() is incorrect", value, _conConnectionSettings.isTcpNodelay());
+    }
+}