Revision: 480
          http://svn.sourceforge.net/stripes/?rev=480&view=rev
Author:   bengunter
Date:     2007-03-03 21:16:25 -0800 (Sat, 03 Mar 2007)

Log Message:
-----------
Resolved STS-315: MockHttpServletRequest.addHeader and getHeader do not work 
with upper-case header names

Modified Paths:
--------------
    trunk/stripes/src/net/sourceforge/stripes/mock/MockHttpServletRequest.java
    trunk/tests/src/net/sourceforge/stripes/mock/TestMockRoundtrip.java

Modified: 
trunk/stripes/src/net/sourceforge/stripes/mock/MockHttpServletRequest.java
===================================================================
--- trunk/stripes/src/net/sourceforge/stripes/mock/MockHttpServletRequest.java  
2007-03-04 05:10:48 UTC (rev 479)
+++ trunk/stripes/src/net/sourceforge/stripes/mock/MockHttpServletRequest.java  
2007-03-04 05:16:25 UTC (rev 480)
@@ -116,6 +116,8 @@
 
     /** Returns any header as a String if it exists. */
     public String getHeader(String name) {
+        if (name != null)
+            name = name.toLowerCase();
         Object header = this.headers.get(name);
         if (header != null) {
             return header.toString();

Modified: trunk/tests/src/net/sourceforge/stripes/mock/TestMockRoundtrip.java
===================================================================
--- trunk/tests/src/net/sourceforge/stripes/mock/TestMockRoundtrip.java 
2007-03-04 05:10:48 UTC (rev 479)
+++ trunk/tests/src/net/sourceforge/stripes/mock/TestMockRoundtrip.java 
2007-03-04 05:16:25 UTC (rev 480)
@@ -217,4 +217,11 @@
         Assert.assertEquals(trip.getDestination(), "/mock/success.jsp");
         Assert.assertEquals(trip.getRequest().getAttribute("integerResult"), 
new Integer(2));
     }
+    
+    @Test(groups="fast")
+    public void testRequestCaseInsensitive(){
+       MockHttpServletRequest request = new MockHttpServletRequest("", "");
+       request.addHeader("User-Agent", "Netscape/6.0");
+       Assert.assertEquals(request.getHeader("User-Agent"), "Netscape/6.0", 
MockHttpServletRequest.class + ".addHeader/getHeader do not properly");
+    }
 }


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development

Reply via email to