On Wed, 2008-07-16 at 04:28 +0100, sebb wrote: > On 15/07/2008, Tom Cellucci <[EMAIL PROTECTED]> wrote: > > Hello, > > > > I was recently working on a project where we wanted to run JMeter from our > > Maven build process. At that time we installed the existing JMeterMojo, > > tried it, and ended up making a few changes. My client has graciously > > offered to contribute these changes back to the JMeter project, if you have > > interest. > > Thanks! > > > Specifically these changes are: > > > > (1) enhanced JMeterMojo that sets up a context class loader based on the > > Maven project's test classpath (allows JMeter test classes to exist within > > the maven project under test) > > The JMeter project does not currently use Maven and has no Maven > add-ons, so it would not be easy to maintain/test. > > Perhaps your changes could be contributed back to the original > JMeterMojo authors?
Sebastian If you ever decide to port JMeter to Maven I will be willing to give you a helping hand with that. We could still use the existing scripts to build the distribution packages and generate the web site. It would be beneficial, though, at least in my opinion, to use Maven to manage dependencies and compile JMeter. Oleg > > > (2) a JMeter-TestNG integration that allows TestNG tests to run as part of > > a > > JMeter test (supports ThreadGroup iterations, manages TestNG lifecycle > > callbacks, etc) > > This does sound like a useful addition to JMeter - there is already a > JUnit sampler. > > > Please let me know whether you would be interested in contribution of these > > changes, and if so, how to proceed. Thank you, > > Yes, we would be interested in the JMeter-TestNG addition. > > I assume this is a reasonably substantial piece of code, so it would > be useful if the contributor(s) could provide CLAs: > > http://www.apache.org/licenses/#clas > > The code itself can be provided as an attachment to a Bugzilla issue. > Please ensure that the appropriate Apache License headers are present > in all source files. > > Thanks! > > > > > > -Tom > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
