[jira] Commented: (MNG-441) surefire plugin needs to be able to fork tests
[ http://jira.codehaus.org/browse/MNG-441?page=comments#action_52127 ] Joakim Erdfelt commented on MNG-441: Additional note. This is required for the Cobertura plugin to work properly. surefire plugin needs to be able to fork tests -- Key: MNG-441 URL: http://jira.codehaus.org/browse/MNG-441 Project: Maven 2 Type: Improvement Components: maven-surefire-plugin Reporter: Brett Porter Fix For: 2.0.1 Attachments: SurefirePlugin.java, pom.xml they can leak memory, so a fork once for all option would be good. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/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]
[jira] Commented: (MNG-441) surefire plugin needs to be able to fork tests
[ http://jira.codehaus.org/browse/MNG-441?page=comments#action_52064 ] Mark Proctor commented on MNG-441: -- Is that .java and .pom working? If so any chance we could have a surefire snapshot release? We are currently blocked from moving Drools to M2 as maven tests for jython fail if we can't fork. surefire plugin needs to be able to fork tests -- Key: MNG-441 URL: http://jira.codehaus.org/browse/MNG-441 Project: Maven 2 Type: Improvement Components: maven-surefire-plugin Reporter: Brett Porter Fix For: 2.0.1 Attachments: SurefirePlugin.java, pom.xml they can leak memory, so a fork once for all option would be good. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/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]
[jira] Commented: (MNG-441) surefire plugin needs to be able to fork tests
[ http://jira.codehaus.org/browse/MNG-441?page=comments#action_51688 ] David Jackman commented on MNG-441: --- One feature of forking the tests that we're currently using in M1 is the working directory for the test process (maven.junit.dir). It's possible we can accomplish what we need to do by setting environment variables, but for completeness the working directory ought to be included. surefire plugin needs to be able to fork tests -- Key: MNG-441 URL: http://jira.codehaus.org/browse/MNG-441 Project: Maven 2 Type: Improvement Components: maven-surefire-plugin Reporter: Brett Porter Fix For: 2.0.1 Attachments: SurefirePlugin.java, pom.xml they can leak memory, so a fork once for all option would be good. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/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]
[jira] Commented: (MNG-441) surefire plugin needs to be able to fork tests
[ http://jira.codehaus.org/browse/MNG-441?page=comments#action_48411 ] Dave Sag commented on MNG-441: -- Hi Andy - thanks for that. Am bearing with you. also note this issue x-references issue MNG-1087. surefire plugin needs to be able to fork tests -- Key: MNG-441 URL: http://jira.codehaus.org/browse/MNG-441 Project: Maven 2 Type: Improvement Components: maven-surefire-plugin Reporter: Brett Porter Fix For: 2.0-beta-4 Attachments: SurefirePlugin.java, pom.xml they can leak memory, so a fork once for all option would be good. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/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]
[jira] Commented: (MNG-441) surefire plugin needs to be able to fork tests
[ http://jira.codehaus.org/browse/MNG-441?page=comments#action_48335 ] Andy Glick commented on MNG-441: Dave - If you take a look at Brett's remarks of Sep 22, 2005, you'll notice that he mentions jvmargs as a required addition to the supported surefire argument list. In a complete surefire/surefire plugin implementation you would add any ea arguments to the jvmargs and you wouldn't have to use MAVEN_OPTS. I actually do sympathize with your desire to see this completed, as it was my desire to see it completed that lead me to attempt to add the forking behavior. :-) There is still some unfinished business WRT forking, but we are going to get to jvmargs, please bear with us. surefire plugin needs to be able to fork tests -- Key: MNG-441 URL: http://jira.codehaus.org/browse/MNG-441 Project: Maven 2 Type: Improvement Components: maven-surefire-plugin Reporter: Brett Porter Fix For: 2.0-beta-4 Attachments: SurefirePlugin.java, pom.xml they can leak memory, so a fork once for all option would be good. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/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]
[jira] Commented: (MNG-441) surefire plugin needs to be able to fork tests
[ http://jira.codehaus.org/browse/MNG-441?page=comments#action_48189 ] Dave Sag commented on MNG-441: -- I'd really like to see the surefire plugin support the -enableassertions (aka -ea) flag too. Right now I have to set MAVEN_OPTS=-ea in my profile but this is not portable. As a result unit tests that pass fine on my machine will fail on any other developers machine unless they too have remembered to set the MAVEN_OPTS environment variable correctly. surefire plugin needs to be able to fork tests -- Key: MNG-441 URL: http://jira.codehaus.org/browse/MNG-441 Project: Maven 2 Type: Improvement Components: maven-surefire-plugin Reporter: Brett Porter Fix For: 2.0-beta-4 Attachments: SurefirePlugin.java, pom.xml they can leak memory, so a fork once for all option would be good. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/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]
[jira] Commented: (MNG-441) surefire plugin needs to be able to fork tests
[ http://jira.codehaus.org/browse/MNG-441?page=comments#action_48172 ] Andy Glick commented on MNG-441: Keep realizing that being new to all of these tools leads me to make numerous mistakes. I have produced and checked into svn code that forks surefire tests or runs them in process. The code is in the surefire branch SUREFIRE_FORK_EXEC. I copied the newest Java report classes into the branch. All of the artifacts created from the branch will have version numbers that should not collide with any other versions, neither now or in the future. So it should be easy to checkout the surefire branch and build and deploy its artifacts. I've uploaded the Java code and a modified POM for a version of the surefire plugin that is compatible with the surefire changes. To fork the surefire tests you add a forkModeonce/forkMode to the surefire plugin's configuration. The plugin's forkMode value defaults to none. Brett suggested, and I intend to implement, a monitor implementation using a listener interface with concrete implmentation classes. That way clients can hook up the surefire component's logging messages if they choose. surefire plugin needs to be able to fork tests -- Key: MNG-441 URL: http://jira.codehaus.org/browse/MNG-441 Project: Maven 2 Type: Improvement Components: maven-surefire-plugin Reporter: Brett Porter Fix For: 2.0-beta-4 Attachments: SurefirePlugin.java, pom.xml they can leak memory, so a fork once for all option would be good. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/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]
[jira] Commented: (MNG-441) surefire plugin needs to be able to fork tests
[ http://jira.codehaus.org/browse/MNG-441?page=comments#action_47633 ] Andy Glick commented on MNG-441: This is progressing - I have hacked a way (on windows) to fork Surefire tests once and have successfully executed the tests from the surefire subproject using the surefire-booter to fork the Surefire class - the reason for the hack is that there is a problem with plexus-utils - it seems that under windows, which i am using, classpaths with embedded spaces cause runtime.exec to fail in addition I'm not driving this with the surefire plugin, but am emulating the surefire plugin. the emulator calls the SurfireBooter class with the same arguments + forkMode aas the plugin i have filed PLX-161 which reports the plexus-utils issue surefire plugin needs to be able to fork tests -- Key: MNG-441 URL: http://jira.codehaus.org/browse/MNG-441 Project: Maven 2 Type: Improvement Components: maven-surefire-plugin Reporter: Brett Porter Fix For: 2.0-beta-3 they can leak memory, so a fork once for all option would be good. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/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]
[jira] Commented: (MNG-441) surefire plugin needs to be able to fork tests
[ http://jira.codehaus.org/browse/MNG-441?page=comments#action_47393 ] Andy Glick commented on MNG-441: Brett, what about maven.junit.usefile = false? surefire plugin needs to be able to fork tests -- Key: MNG-441 URL: http://jira.codehaus.org/browse/MNG-441 Project: Maven 2 Type: Improvement Components: maven-surefire-plugin Reporter: Brett Porter Fix For: 2.0-beta-3 they can leak memory, so a fork once for all option would be good. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/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]
[jira] Commented: (MNG-441) surefire plugin needs to be able to fork tests
[ http://jira.codehaus.org/browse/MNG-441?page=comments#action_47399 ] Brett Porter commented on MNG-441: -- that's in another jira task... those were the ones related to forking. How is this progressing? surefire plugin needs to be able to fork tests -- Key: MNG-441 URL: http://jira.codehaus.org/browse/MNG-441 Project: Maven 2 Type: Improvement Components: maven-surefire-plugin Reporter: Brett Porter Fix For: 2.0-beta-3 they can leak memory, so a fork once for all option would be good. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/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]
[jira] Commented: (MNG-441) surefire plugin needs to be able to fork tests
[ http://jira.codehaus.org/browse/MNG-441?page=comments#action_46956 ] Brett Porter commented on MNG-441: -- options we need to support (from m1): - fork mode - jvmargs - envvars - jvm location surefire plugin needs to be able to fork tests -- Key: MNG-441 URL: http://jira.codehaus.org/browse/MNG-441 Project: Maven 2 Type: Improvement Components: maven-surefire-plugin Reporter: Brett Porter Fix For: 2.0-beta-2 they can leak memory, so a fork once for all option would be good. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/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]
[jira] Commented: (MNG-441) surefire plugin needs to be able to fork tests
[ http://jira.codehaus.org/browse/MNG-441?page=comments#action_46661 ] Andy Glick commented on MNG-441: I've started work on this. Will add a class ForkMode as an enumerated type in surefire and the plugin will reference it. The surefirebooter class will determine the ForkMode value passed by the plugin {NONE, ONCE, EACH_TEST} default value will be ONCE and kick off tests according to ForkMode - I intend to add a class SurefireConfiguration to make this similar to plexus.compiler, I'm attempting to create a protocol for parameter passing as we've already got surefire-beanshell, surefire-jython, surefire-web and surefire-xmlrpc, i expect to add surefire-testng and possibly surefire-ivalidator when things settle down a bit :) surefire plugin needs to be able to fork tests -- Key: MNG-441 URL: http://jira.codehaus.org/browse/MNG-441 Project: Maven 2 Type: Improvement Components: maven-surefire-plugin Reporter: Brett Porter Fix For: 2.0-beta-2 they can leak memory, so a fork once for all option would be good. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/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]
[jira] Commented: (MNG-441) surefire plugin needs to be able to fork tests
[ http://jira.codehaus.org/browse/MNG-441?page=comments#action_46470 ] Brett Porter commented on MNG-441: -- I agree with doing it in surefire. I didn't think it executed them in separate threads though (just in a separate classloader). I will reply on the surefire list. surefire plugin needs to be able to fork tests -- Key: MNG-441 URL: http://jira.codehaus.org/browse/MNG-441 Project: Maven 2 Type: Improvement Components: maven-surefire-plugin Reporter: Brett Porter Fix For: 2.0-beta-2 they can leak memory, so a fork once for all option would be good. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/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]
[jira] Commented: (MNG-441) surefire plugin needs to be able to fork tests
[ http://jira.codehaus.org/browse/MNG-441?page=comments#action_46387 ] Andy Glick commented on MNG-441: Is the intention to have M2 do the forking, or will the forking behavior be added to surefire, and be passed from the plugin to surefire by parameter? That seems like a better separation of concerns. The differences between surefire and the JUnit Ant Task are pretty significant, and surefire's native mode is to run tests in separate threads, using an isolated class loader so it might be a good idea to get the surefire guys input concerning forking the test cases. surefire plugin needs to be able to fork tests -- Key: MNG-441 URL: http://jira.codehaus.org/browse/MNG-441 Project: Maven 2 Type: Improvement Components: maven-plugins Reporter: Brett Porter Fix For: 2.0-beta-2 they can leak memory, so a fork once for all option would be good. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/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]
[jira] Commented: (MNG-441) surefire plugin needs to be able to fork tests
[ http://jira.codehaus.org/browse/MNG-441?page=comments#action_43645 ] Vincent Massol commented on MNG-441: Note: This is required for the Clover plugin to work properly surefire plugin needs to be able to fork tests -- Key: MNG-441 URL: http://jira.codehaus.org/browse/MNG-441 Project: Maven 2 Type: Improvement Components: maven-plugins Reporter: Brett Porter Fix For: 2.0-beta-2 they can leak memory, so a fork once for all option would be good. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/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]