[GitHub] [tinkerpop] vkagamlyk commented on pull request #2052: Disable Parallelization for Gherkin tests

["vkagamlyk (via GitHub)" <gi...@apache.org>]

vkagamlyk commented on PR #2052:
URL: https://github.com/apache/tinkerpop/pull/2052#issuecomment-1533220257

   > Just for my own understanding: The problem here is not that some Gherkin tests are executed in parallel which shouldn't happen with xUnit as tests within the same class [are executed sequentially](https://xunit.net/docs/running-tests-in-parallel), but that other integration tests are executed in parallel with the Gherkin tests, right? And you only add this config to the Gherkin tests and not also to those other integration tests, because it's currently enough if we execute the Gherkin tests completely sequentially?
   
   Yes, some Gherkin tests temporary add properties to Modern graph and this leads to random errors. [For example](https://github.com/apache/tinkerpop/actions/runs/4833390970/jobs/8613413406). The probability of an error on the master is much greater because tests for properties have been added there.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org