Updated Branches: refs/heads/flume-1.4 c587e0688 -> a7b9b7eb3
FLUME-2026. TestHTTPSource should use any available port rather than a hardcoded port number. (Hari Shreedharan, Mike Percy via Mike Percy) Project: http://git-wip-us.apache.org/repos/asf/flume/repo Commit: http://git-wip-us.apache.org/repos/asf/flume/commit/a7b9b7eb Tree: http://git-wip-us.apache.org/repos/asf/flume/tree/a7b9b7eb Diff: http://git-wip-us.apache.org/repos/asf/flume/diff/a7b9b7eb Branch: refs/heads/flume-1.4 Commit: a7b9b7eb3b19281a22d8fa914e4e34bb58520624 Parents: c587e06 Author: Mike Percy <[email protected]> Authored: Sun Jun 23 04:08:46 2013 -0700 Committer: Mike Percy <[email protected]> Committed: Sun Jun 23 04:10:19 2013 -0700 ---------------------------------------------------------------------- .../apache/flume/source/http/TestHTTPSource.java | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/flume/blob/a7b9b7eb/flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java ---------------------------------------------------------------------- diff --git a/flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java b/flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java index 59d1cb1..8952db3 100644 --- a/flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java +++ b/flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java @@ -44,6 +44,7 @@ import org.junit.Test; import javax.servlet.http.HttpServletResponse; import java.io.IOException; import java.lang.reflect.Type; +import java.net.ServerSocket; import java.util.ArrayList; import java.util.List; import java.util.Map; @@ -58,12 +59,21 @@ public class TestHTTPSource { private static HTTPSource source; private static Channel channel; - private int selectedPort; + private static int selectedPort; DefaultHttpClient httpClient; HttpPost postRequest; + private static int findFreePort() throws IOException { + ServerSocket socket = new ServerSocket(0); + int port = socket.getLocalPort(); + socket.close(); + return port; + } + @BeforeClass public static void setUpClass() throws Exception { + selectedPort = findFreePort(); + source = new HTTPSource(); channel = new MemoryChannel(); @@ -82,7 +92,7 @@ public class TestHTTPSource { channel.start(); Context context = new Context(); - context.put("port", String.valueOf(41404)); + context.put("port", String.valueOf(selectedPort)); context.put("host", "0.0.0.0"); Configurables.configure(source, context); @@ -98,7 +108,7 @@ public class TestHTTPSource { @Before public void setUp() { httpClient = new DefaultHttpClient(); - postRequest = new HttpPost("http://0.0.0.0:41404";); + postRequest = new HttpPost("http://0.0.0.0:"; + selectedPort); } @Test
