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

Feature suggestion: Freeze files and stored.rules validation / sanity check #1264

Open
keiki85 opened this issue Mar 12, 2024 · 1 comment
Open

Comments

@keiki85
Copy link

keiki85 commented Mar 12, 2024

Hi ArchUnit Team,

First of you do a great job here. ArchUnit is great.

In our project we currently have >70 freeze files in the stored.rules mentioned.

Digging deeper I noticed several things, which I am concerned about.

  1. The files referenced might not exist anymore, but the reference in stored.rules is
  2. When freeze files become empty you are not being advised to remove the freezing from your test.
  3. Freeze files might still be in the archunit folder, but not referenced anymore
  4. Tests were removed without removing the stored.rules line or freeze file
  5. stored.rules are not sorted. To make it better human readable it would be nice to have it sorted alphabetically.

Of course in the end it is our job to keep our source code in check and clean up correctly etc.

Yet I wonder how ArchUnit itself can help to have a sanity check of its freeze files and stored.rules?

What is your opinion on this?

@codecholeric
Copy link
Collaborator

Sorry for the late reply, I think it would be a nice addition, if anybody wants to give this a shot I would support it! In the past we've already sorted the actual violations alphabetically to ease human browsing and diffs if I remember correctly.

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

No branches or pull requests

2 participants