[
https://issues.apache.org/jira/browse/CACTUS-235?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12574295#action_12574295
]
Petar Tahchiev commented on CACTUS-235:
---------------------------------------
Hi Jeff,
Thanks for the patch - I have already applied it in the trunk.
> TestShareAll suite adds TestHttpSession tests incorrectly
> ---------------------------------------------------------
>
> Key: CACTUS-235
> URL: https://issues.apache.org/jira/browse/CACTUS-235
> Project: Cactus
> Issue Type: Bug
> Components: Servlet Sample
> Affects Versions: 1.7.1
> Environment: Windows XP, WSAD 5.1.2
> Reporter: Jeff Glass
> Priority: Minor
> Fix For: 1.8
>
>
> 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.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
