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/12 12:12:11 UTC

maven-surefire pull request: forkMode onceperthread (SUREFIRE-751)

GitHub user agudian opened a pull request:

    https://github.com/apache/maven-surefire/pull/8

    forkMode onceperthread (SUREFIRE-751)

    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
    
    Open points: 
    - Source-code formatting, Apache file headers missing
    - Some Javadoc here and there... I'll add that soon.
    - refactoring of the forkMode options to simplify the usage? See my proposals in the above mailing list thread... :)
    - anything else? Let me know! :)

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/agudian/maven-surefire fm-onceperthread

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/maven-surefire/pull/8.patch

----
commit a9f1c138464acd7762ccc3871672843696b0cc57
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).

commit ba2e51f215cbce21ee9833b13042466c7358f4ef
Author: agudian <an...@gmail.com>
Date:   2012-11-07T21:13:16Z

    Don't use the extra thread for reading the next test class

commit cbb158aeecd90c2e7b61f5496a83a79018737421
Author: agudian <an...@gmail.com>
Date:   2012-11-11T20:37:34Z

    [SUREFIRE-924] deploy Surefire plugins site in /surefire instead of
    /plugins
    [SUREFIRE-913] forkMode always - RejectedExecutionException when > 500 tests
    
    Fix as suggested by Garry Tully

commit 23c6caa76bca4905200ef1f3185d48a9c9939850
Author: agudian <an...@gmail.com>
Date:   2012-11-11T20:40:15Z

    support of forkMode=onceperthread in TestNG provider

commit 27057cec3daebe73be28f39aeb12950588014e9b
Author: agudian <an...@gmail.com>
Date:   2012-11-11T23:05:30Z

    Adapt and add integration tests for forkMode=onceperthread

commit 0660c167ed83ec1cff4ba1e2f3f4b039cbb92fee
Author: agudian <an...@gmail.com>
Date:   2012-11-12T10:52:37Z

    fix gathering test results with forkMode=onceperthread (fixes faild ITs)

----


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org