[WW-1950] UrlHelper.buildUrl does not output port even if forceAddSchemeHostAndPort turned on (TestCase included)

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.8
• Fix Version/s: 2.0.10, 2.1.0
• Component/s: Plugin - Tags
• Flags: Important
• Assignee: James Holmes
• Reporter: Sami Dalouche
• Created: 27/May/07 16:44
• Updated: 28/Oct/12 22:07
• Resolved: 14/Aug/07 02:08

Description

If you add this test to UrlHelperTest, you will find out that it fails..

public void testForceAddSchemeHostAndPortWithNonStandardPort() throws Exception

{ String expectedUrl = "http://localhost:9090/contextPath/path1/path2/myAction.action"; Mock mockHttpServletRequest = new Mock(HttpServletRequest.class); mockHttpServletRequest.expectAndReturn("getScheme", "http"); mockHttpServletRequest.expectAndReturn("getServerName", "localhost"); mockHttpServletRequest.expectAndReturn("getContextPath", "/contextPath"); mockHttpServletRequest.expectAndReturn("getServerPort", 9090); Mock mockHttpServletResponse = new Mock(HttpServletResponse.class); mockHttpServletResponse.expectAndReturn("encodeURL", expectedUrl, expectedUrl); String result = UrlHelper.buildUrl("/path1/path2/myAction.action", (HttpServletRequest) mockHttpServletRequest.proxy(), (HttpServletResponse)mockHttpServletResponse.proxy(), null, "http", true, true, true); assertEquals(expectedUrl, result); mockHttpServletRequest.verify(); }