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

RAT-98: report skipped files #417

Draft
wants to merge 50 commits into
base: master
Choose a base branch
from

Conversation

Claudenw
Copy link
Contributor

@Claudenw Claudenw commented Dec 26, 2024

Fixes issues raised by report skipped processing, specifically issues with Maven not correctly skipping files that should be skipped.

In the process of working on the bugs additional testing code was implemented to allows testing on file systems that are not native. Most name based code now checks against UNIX, WINDOWS, and OSX.

Significant additional work was undertaken to ensure that the document names work as expected.

@ottlinger ottlinger changed the title Rat 98 report skipped files RAT-98: report skipped files Dec 26, 2024
@ottlinger
Copy link
Contributor

@Claudenw did you accidentally push an old branch as the build fails with spotbugs errors?

@Claudenw
Copy link
Contributor Author

@ottlinger
Actually, it is a work in progress. I had to rework the descending exclusion processing because it was returning incorrect results. That triggered a number of changes in integration tests and the like because they were not set correctly to begin with. Net result is a large change but accurate results and more use of the assertJ package.

My next step is to see how I can split this into smaller pieces.

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

Successfully merging this pull request may close these issues.

2 participants