Tibor17 commented on issue #190: [SUREFIRE-1535] Surefire unable to run testng 
suites in parallel
URL: https://github.com/apache/maven-surefire/pull/190#issuecomment-404166769
 
 
   @andreybpanfilov 
   Pls explore the following.
   I expected these two `FORKTESTSET`, `TEST_SUITE_XML_FILES` been exclusive. 
Not sure if this is a fix or a workaround because in this commit we passed the 
value from variable `testSet` into `properties.addList`.
   Question, should be the `testSuiteDefinition.getSuiteXmlFiles()` be already 
filled up with value and then `testSet` become null? Here we interchanged these 
two, but mabe the (pre)caller of BooterSerializer should detect this situation 
with XML Suites.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to