maven-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Tibor17 <>
Subject [GitHub] maven-surefire pull request: Quick fix for SUREFIRE-1177 and SUREF...
Date Fri, 18 Sep 2015 15:34:27 GMT
Github user Tibor17 commented on the pull request:
    LGTM but missing ITs.
    You can use the one in Jira and create new for testsuites. See how the ITs are written
in surefire-integration-tests module. I guess you already have the experiences. 
    We have to update documentation regarding parallel exec in TestNG with use of these parameters
and trivial use cases for each.
    As a hint for `dataproviderthreadcount` you can print `ConcurrentHashMap#size()` and assert
that string `concurrency=30.` appears in console lines. You would need to put a little delay
100 millis where iterations 1000 would be enough to have.
    This way you do not need any performance test - no need to assert exec time which is always
dependent on CPU in build park.
    Regarding `suitethreadpoolsize` is there any way to print verbose configuration by TestNG?
    Other possibility would be to implement `ITestListener` remember time of the first `onStart`
and time of last `oninish` and compute the diff time. This would be the performance test,
but it may vary 1.0 sec in overloaded build park.
    One way or another the IT should assert that all suites run successfully.

If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message