search

allure-framework / allure-cucumberjvm

Deprecated, use https://github.com/allure-framework/allure-java instead
Apache License 2.0
16 stars 13 forks source link

Add synchronozation to list operations #39

Closed clicman closed 7 years ago

clicman commented 7 years ago

Fixes #38

baev commented 7 years ago

Is there any way to write the test to reproduce the problem?

baev commented 7 years ago

This test found the same problem in allure1 https://github.com/allure-framework/allure1/blob/master/allure-java-adaptor-api/src/test/java/ru/yandex/qatools/allure/AllureLifecycleTest.java#L186-L220

clicman commented 7 years ago

There shouldn't be reproducable, because cucumber-jvm test supports classes only threading. And in this case for each class new Plugin instance initiated. So I have no idea how it possible.

baev commented 7 years ago

Hm.. Is it possible to manually parallelize steps during test?

clicman commented 7 years ago

Yes, it possible though pointing <parallel>methods</parallel> in surefire config, but it will crash cucumber-jvm itself on its not synchronized internal lists and it will happen before original problem. As I said cucumber-jvm supports classes parallelization only.

baev commented 7 years ago

I got it, thanks.