[ 
https://issues.apache.org/jira/browse/SUREFIRE-1177?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14875821#comment-14875821
 ] 

ASF GitHub Bot commented on SUREFIRE-1177:
------------------------------------------

Github user juherr commented on the pull request:

    https://github.com/apache/maven-surefire/pull/105#issuecomment-141488763
  
    I will try to update it next week. What is the deadline if I want the fix 
in the next release?
    
    > See how the ITs are written in surefire-integration-tests module. I guess 
you already have the experiences.
    
    Not yet, but I can try.
    Could you point me a current IT test that is a good example in our context?
    
    > If you have exps with overriding git commit, feel free to override it 
adding the ITs 
    
    Ok, will do. And regarding the fix, it is more SUREFIRE-1179 than 
SUREFIRE-1177, so I will update the commit message too (or do you prefer 
another issue for the support of the "new" testng configuration method?).
    



> TestNG "suitethreadpoolsize" parameter can not be set by Maven Surefire
> -----------------------------------------------------------------------
>
>                 Key: SUREFIRE-1177
>                 URL: https://issues.apache.org/jira/browse/SUREFIRE-1177
>             Project: Maven Surefire
>          Issue Type: Bug
>          Components: TestNG support
>    Affects Versions: 2.18.1
>            Reporter: Jan Dundáček
>
> Because of Surefire is using deprecated TestNG.configure(Map) method, 
> parameter "suitethreadpoolsize" value is not read by TestNG. And so test 
> suites can not be run in parallel. This parameter is read only in 
> TestNG.configure(CommandLineArgs) method.
>   /**
>    * This method is invoked by Maven's Surefire to configure the runner,
>    * do not remove unless you know for sure that Surefire has been updated
>    * to use the new configure(CommandLineArgs) method.
>    *
>    * @deprecated use new configure(CommandLineArgs) method
>    */
>   @Deprecated
>   public void configure(Map cmdLineArgs) {
>   ....



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to