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

Steps are being displayed as tests #21

Closed Jiwari closed 7 years ago

Jiwari commented 8 years ago

The report is displaying every step as a "test", this creates confusion as there can be displayed 200 tests, when there actually only 20 cucumber Scenarios.

I'm not sure if this is suppose to work like this, but my vision is that the report should display on the graphs the cucumber "Scenarios" that failed/passed/broken instead of each separate stepdefinition.

clicman commented 8 years ago

I agree with you but this behaviour comes from cucumber-jvm-junit plugin.

clicman commented 8 years ago

I decided to reopen this issue. I have some thouts how to implement it.

clicman commented 8 years ago

@baev I need some support to analyze it. My implimentation plan is to replace suiteStarted/Finished/.. events by TestCaseStarted/Finished/... and replace TestCaseStarted/Finished/.. by StepStarted/Finished/.. But as I understand, I cannot fully manage steps statuses - only last one. It will cause multithreading issues. Any Ideas how to deal with it? Maybe is it possible to expose steps management api in core as it for test cases for example, to magange steps by some unique identifier?

clicman commented 8 years ago

As alternative it can be fixed via cucumber-jvm patching. See https://github.com/cucumber/cucumber-jvm/pull/588 for details.

clicman commented 7 years ago

Will be fixed while PR#36 merged

nicolas-duvivier commented 7 years ago

That's nice ! Good job

clicman commented 7 years ago

Fixed in master. Will available in 1.6 release