Skip to content

Add exclude-test filter flag#3033

Open
TUPM96 wants to merge 1 commit into
crytic:masterfrom
TUPM96:add-exclude-test-flag
Open

Add exclude-test filter flag#3033
TUPM96 wants to merge 1 commit into
crytic:masterfrom
TUPM96:add-exclude-test-flag

Conversation

@TUPM96
Copy link
Copy Markdown

@TUPM96 TUPM96 commented May 25, 2026

Summary

  • Add a --exclude-test detector flag that appends a default test/mock path filter.
  • Allow combining it with existing --filter-paths, while rejecting --include-paths because include filtering takes the opposite path.
  • Cover the helper and default pattern with unit tests.

Fixes #691

Testing

  • uv run pytest tests/unit/utils/test_command_line.py
  • uv run ruff check slither/main.py slither/utils/command_line.py tests/unit/utils/test_command_line.py
  • codespell slither/main.py slither/utils/command_line.py tests/unit/utils/test_command_line.py
  • git diff --check

Copilot AI review requested due to automatic review settings May 25, 2026 08:00
@TUPM96 TUPM96 requested a review from smonicas as a code owner May 25, 2026 08:00
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

Add --exclude-test flag

3 participants