TestShareAll suite adds TestHttpSession tests incorrectly
---------------------------------------------------------

         Key: CACTUS-235
         URL: http://issues.apache.org/jira/browse/CACTUS-235
     Project: Cactus
        Type: Bug
  Components: Servlet Sample  
    Versions: 1.7.1    
 Environment: Windows XP, WSAD 5.1.2
    Reporter: Jeff Glass
    Priority: Minor


The method org.apache.cactus.sample.servlet.unit.TestShareAll.suite() adds the 
TestHttpSession tests to the suite incorrectly. It doesn't call 
TestHttpSession.suite(), so the tests are not properly set up, and as a result 
TestHttpSession.beginDependentTestUsingSession2(WebRequest) throws a 
null-pointer exception (because this.dependentTest is null).

I found this problem when running the Cactus sample tests through a web browser 
(http://localhost:9080/testweb/ServletTestRunner?suite=org.apache.cactus.sample.servlet.unit.TestShareAll).

The patch follows.

/jeff

--- TestShareAll.java.orig      2006-01-06 14:54:00.663897700 -0500
+++ TestShareAll.java   2006-01-06 14:54:32.600776000 -0500
@@ -55,7 +55,7 @@
         suite.addTestSuite(TestHttpUnitIntegration.class);
         suite.addTestSuite(TestServletRedirectorOverride.class);
         suite.addTestSuite(TestHttpParameters.class);
-        suite.addTestSuite(TestHttpSession.class);
+        suite.addTest(TestHttpSession.suite());
         suite.addTestSuite(TestHttpResponse.class);
         suite.addTestSuite(TestCookie.class);
         suite.addTestSuite(TestRequestDispatcher.class);


-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to