You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by agudian <gi...@git.apache.org> on 2012/11/16 23:11:25 UTC
maven-surefire pull request: forkMode onceperthread (SUREFIRE-751) (licence...
GitHub user agudian opened a pull request:
https://github.com/apache/maven-surefire/pull/10
forkMode onceperthread (SUREFIRE-751) (licence headers included)
In relation to http://jira.codehaus.org/browse/SUREFIRE-751
Adds a new forkMode option "onceperthread", that creates one reusable forked process for each thread specified with the threadCount option. The tests are handed out to the processes "on-demand", i.e. as soon as a test class was executed, a new class to execute is supplied by the parent process.
Adds a resulotion of the placeholder ${surefire.threadNumber} to the number of the executing thread (range [1..threadCount]). The resolution takes place in system properties in the argLine. E.g. -DmyDatabaseSchema=TEST_SCHEMA_${surefire.threadNumber}. Works for all forkModes (including "never", which would resolve the placeholder to "1").
Integration Tests ForkModeIT and ForkModeNGIT have been extended to cover both new features
Discussion in http://markmail.org/message/xn7bo4xxnzpvmskw
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/agudian/maven-surefire fm-onceperthread-rebased
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/maven-surefire/pull/10.patch
----
commit efff9ff7ea46a57682a54802fa127237c1842ba8
Author: agudian <an...@gmail.com>
Date: 2012-11-06T20:54:00Z
forkMode=onceperthread, working for JUnit providers - TestNG support sill needs some work. Requires maven-shared-utils version 0.2 (or latest snapshot of it).
Don't use the extra thread for reading the next test class
support of forkMode=onceperthread in TestNG provider
Adapt and add integration tests for forkMode=onceperthread
fix gathering test results with forkMode=onceperthread (fixes faild ITs)
commit 996c65a5fd25a43426461a5cae4e2809c1f5459a
Author: agudian <an...@gmail.com>
Date: 2012-11-16T22:05:14Z
Merge pull request #1 from krosenvold/master
Need a couple of license headers
commit f341dff9da23d5a2ae4ac6ffa1eb8dd8e27a855c
Author: agudian <an...@gmail.com>
Date: 2012-11-16T22:09:52Z
Added licence headers, javadoc, did some minor renaming
----
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org