[jira] Commented: (SHIRO-93) Add container-based integration tests for samples/web module

["Kalle Korhonen (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SHIRO-93?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12750140#action_12750140 ] 

Kalle Korhonen commented on SHIRO-93:
-------------------------------------

Note that the test fails (at least on my machine) because "http://java.sun.com/jsp/jstl/core cannot be resolved in either web.xml or the jar files deployed with this application". The same test succeeded in Eclipse as m2eclipse imported project. The root cause is that there are some minor differences between Maven compile classpath and surefire classpath, but both jstl and standard libs were there (could be just the order of the lib, perhaps some lib is declaring an older jstl uri?). Anyway, I didn't feel like investigating further and instead I configured surefire to not fork as follows:

						<plugin>
							<artifactId>maven-surefire-plugin</artifactId>
							<configuration>
								<forkMode>never</forkMode>
							</configuration>
						</plugin>

This has the added benefit that the tests should run a tiny bit faster.

> Add container-based integration tests for samples/web module
> ------------------------------------------------------------
>
>                 Key: SHIRO-93
>                 URL: https://issues.apache.org/jira/browse/SHIRO-93
>             Project: Shiro
>          Issue Type: Test
>          Components: Sample Apps
>    Affects Versions: 1.0
>         Environment: any
>            Reporter: Kalle Korhonen
>             Fix For: 1.0
>
>         Attachments: container-integration-test.patch, update-jetty-version.patch
>
>
> Patch(es) to follow - the first one includes a base class to derive the test classes and one sample tests. I'll add specific tests for login/logout & remember later.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.