Removing unstable tests

The tests removed from SafeRequestTest were attempting to validate
environment state as well as the unit functionality.  The intent of these
tests has been distributed to a workflow test check
(SecurityWrapperRequestTest), a whitelist validation test
(EsapiWhitelistValidationPatternTester), and tests for the PercentCodec's
impact on input data (PercentCodecStringTest, PercentCodecCharacterTest).

The combination of these tests assert that the pieces that were composing
the removed content will function under more explict and controlled
conditions.  Changes to configuration will nChanges to configuration will
now be revealed closer to the component that is directly impacted.

Author: jeremiahjstacey

src/test/java/org/owasp/esapi/filters/SafeRequestTest.java

@@ -94,67 +94,6 @@ public void testGetQueryStringNull()
 		assertNull(wrappedReq.getQueryString());
 	}
 
-	public void testGetQueryStringNonNull()
-	{
-		MockHttpServletRequest req = new MockHttpServletRequest();
-		SecurityWrapperRequest wrappedReq;
-
-		req.setQueryString("a=b");
-		wrappedReq = new SecurityWrapperRequest(req);
-		assertEquals("a=b",wrappedReq.getQueryString());
-	}
-
-	public void testGetQueryStringNUL()
-	{
-		MockHttpServletRequest req = new MockHttpServletRequest();
-		SecurityWrapperRequest wrappedReq;
-
-		req.setQueryString("a=\u0000");
-		wrappedReq = new SecurityWrapperRequest(req);
-		assertEquals("",wrappedReq.getQueryString());
-	}
-
-	public void testGetQueryStringPercent()
-	{
-		MockHttpServletRequest req = new MockHttpServletRequest();
-		SecurityWrapperRequest wrappedReq;
-
-		req.setQueryString("a=%62");
-		wrappedReq = new SecurityWrapperRequest(req);
-		assertEquals("a=b",wrappedReq.getQueryString());
-	}
-	
-	public void testGetQueryStringPercentNUL()
-	{
-		MockHttpServletRequest req = new MockHttpServletRequest();
-		SecurityWrapperRequest wrappedReq;
-
-		req.setQueryString("a=%00");
-		wrappedReq = new SecurityWrapperRequest(req);
-		assertEquals("a="+Character.MIN_VALUE, wrappedReq.getQueryString());
-	}
-
-	public void testGetQueryStringPercentEquals()
-	{
-		MockHttpServletRequest req = new MockHttpServletRequest();
-		SecurityWrapperRequest wrappedReq;
-
-		req.setQueryString("a=%3d");
-		wrappedReq = new SecurityWrapperRequest(req);
-		assertEquals("a==",wrappedReq.getQueryString());
-	}
-
-	public void testGetQueryStringPercentAmpersand()
-	{
-		MockHttpServletRequest req = new MockHttpServletRequest();
-		SecurityWrapperRequest wrappedReq;
-
-		req.setQueryString("a=%26b");
-		wrappedReq = new SecurityWrapperRequest(req);
-		assertEquals("a=&b",wrappedReq.getQueryString());
-	}
-
-
 	// Test to ensure null-value contract defined by ServletRequest.getParameterNames(String) is met.
 	public void testGetParameterValuesReturnsNullWhenParameterDoesNotExistInRequest() {
 		MockHttpServletRequest request = new MockHttpServletRequest();