Don't add extra suite label in allure-cucumberjs to fix #999

allure-framework / allure-js

Allure integrations for JavaScript test frameworks

https://allurereport.org/

Apache License 2.0

217 stars 112 forks source link

Closed YakovL closed 6 days ago

YakovL commented 3 weeks ago

As described in #999 and comments to it, allure-cucumberjs adds the suite label with value of scenario.name, which makes no sense in both cases:

if a user sets their own suite, they get a duplicate in the report due to 2 suite labels;
otherwise (since scenario names are unique), it creates an excessive level of nesting: suite has the same name as the only element in it, see the screen in the original issue.

CLAassistant commented 3 weeks ago

All committers have signed the CLA.

baev commented 6 days ago

@YakovL thanks!