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

Refactored test output folder name #998

Merged
merged 1 commit into from
Jun 22, 2023

Conversation

mrizzi
Copy link
Member

@mrizzi mrizzi commented Jun 21, 2023

Based on @agoncal's report about rules tests execution:

Sometimes when you run tests matching a short word (eg. mvn clean test -DrunTestsMatching=web) you end up with several reports under target . Then, to find the appropriate one is quite difficult, You need to browse through all of them. When you open the report, most of the reports are named data, but not all of them.

which is reflected in the next screenshot
Screenshot 2023-06-21 at 15 12 17

with the changes in this PR, the same tests execution will create this folders structure:
Screenshot from 2023-06-21 16-21-57

@mrizzi mrizzi changed the title Refectored test output folder name Refactored test output folder name Jun 22, 2023
Copy link
Member Author

@mrizzi mrizzi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is all about tests and the changes are all and just to code for testing.
Since tests are fine, I'm going to merge it.

@mrizzi mrizzi merged commit c8a9309 into windup:master Jun 22, 2023
27 checks passed
@mrizzi mrizzi deleted the refactor-tests-output branch June 22, 2023 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant