cosmetic changes

hrcorval · Nov 7, 2023 · 6c5eeed · 6c5eeed
1 parent ad8b510
commit 6c5eeed
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 9 deletions.
diff --git a/CHANGES.rst b/CHANGES.rst
@@ -4,15 +4,15 @@ Version History
 Version: 3.1.0
 -------------------------------------------------------------------------------
 ENHANCEMENTS:
-* Improvement in background steps rendering in HTML report
-* Reporting scenarios that crashed during execution as "Untested" in HTML report
+
+* Improvement done when rendering feature background steps in HTML report
+* Reporting scenarios that crashed during execution as "Untested" in HTML report (scenarios that crashed were not reported in previous BehaveX versions)
+* Enhancement in HTML Report to add to scenarios the tags related to the feature where they are located
 
 FIXES:
 
-* Fix done to properly report as "Untested" features and scenarios that crashed during execution
 * Fixed console summary, to properly report the number of scenarios executed
 * Fix done when executing features in parallel, as not all features where considered for execution
-* Fix done when executing features in parallel, as not all features where considered for execution
 * Fixed JUnit reports to properly report all executed scenarios (as some of them were missing)
 
 Version: 3.0.0

diff --git a/behavex/outputs/report_xml.py b/behavex/outputs/report_xml.py
@@ -37,16 +37,13 @@ def flatter_scenarios(scenarios_list):
             flatter_scenarios(feature_.scenarios) if isobject else feature_['scenarios']
         )
 
-    def get_tags(scenario_):
-        return get_scenario_tags(scenario_) if isobject else scenario_['tags']
-
     def get_status(scenario_):
         return scenario_.status if isobject else scenario_['status']
 
     scenarios = [
         scenario
         for scenario in get_scenarios(feature)
-        if (match_for_execution(get_tags(scenario)) and not (get_status(scenario) == 'skipped' and get_env('RERUN_FAILURES')))
+        if (match_for_execution(get_scenario_tags(scenario)) and not (get_status(scenario) == 'skipped' and get_env('RERUN_FAILURES')))
     ]
 
     skipped = [scenario for scenario in scenarios if get_status(scenario) == 'skipped']
@@ -59,7 +56,7 @@ def get_status(scenario_):
     muted = [
         scenario
         for scenario in scenarios
-        if any(i in ['MUTE'] for i in get_tags(scenario))
+        if any(i in ['MUTE'] for i in get_scenario_tags(scenario))
     ]
 
     muted_failed = [scenario for scenario in muted if get_status(scenario) == 'failed']