Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Children modules executor.json wrongly overwritten for aggregate report #55

Open
1 of 3 tasks
lalmeras opened this issue Feb 12, 2018 · 3 comments
Open
1 of 3 tasks

Comments

@lalmeras
Copy link

lalmeras commented Feb 12, 2018

I'm submitting a ...

  • bug report
  • feature request
  • support request => Please do not submit support request here, see note at the top of this template.

What is the current behavior?

When aggregate report is used, all executors are reported with the name of the parent module.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem

Create a project with a submodule. submodule must provide test results and activate allure reportSet report; parent module must activate reportSet aggregate.

Run

# generate child
mvn site
# generate aggregate
mvn site

Generated aggregated report show an executor named with the parent module name instead of the child module name.

What is the expected behavior?

Executor should be named with the child module name

Where is the bug

aggregate report (AllureAggregateMojo) triggers AllureGenerateMojo#copyExecutorInfo for each inputDirectories, so for each child module allure-results folder. It overwrites the correctly generated file (generated by report on child module) with a new one with buildName =

I think that copyExecutorInfo should be called only for inputDirectories related to the current module, even when aggregate report is done.

What is the motivation / use case for changing the behavior?

Report consistency

Please tell us about your environment:

  • Allure version: 2.5.0
  • Test framework: junit4 (not test framework related I think)
  • Allure adaptor: allure-junit4 2.0-BETA22
  • Generate report using: [email protected]
@lalmeras
Copy link
Author

Here is an example of broken report (each Igloo - Parent should be a different label):

executors

@lalmeras
Copy link
Author

Any feedback on my report ? If I work on a PR based on my analysis, any chance to see it released ?

@suhail23790
Copy link

Hi any update in this issue, i am not getting Maven and Igloo-Parent, i am creating the executor.json file from the code and I have only 1 attribute in it that is the tester name {"Tester":"ABCD"}, not sure what I am doing wrong but I am getting a hate symbol and name is displayed as "Unknown
Capture
"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants