Skip to content

Add skip test#195

Merged
furtib merged 5 commits intoEricsson:mainfrom
furtib:skip-test
Mar 31, 2026
Merged

Add skip test#195
furtib merged 5 commits intoEricsson:mainfrom
furtib:skip-test

Conversation

@furtib
Copy link
Copy Markdown
Contributor

@furtib furtib commented Feb 13, 2026

Why:
We want to support the option to skip files during analysis in both codechecker rules. This test will help us check whether they are working correctly.

What:

  • Added tests for skipfiles

Addresses:
#160

@furtib furtib requested a review from Szelethus February 13, 2026 12:59
@furtib furtib self-assigned this Feb 13, 2026
@furtib furtib added the test ☑️ Adding or refactoring tests label Feb 13, 2026
Copy link
Copy Markdown
Contributor

@Szelethus Szelethus left a comment

Choose a reason for hiding this comment

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

LGTM after nits

Comment thread test/unit/skip/BUILD Outdated
Comment thread test/unit/skip/BUILD Outdated
Comment thread test/unit/skip/test_skip.py
Comment thread test/unit/skip/test_skip.py
Comment thread test/unit/skip/test_skip.py
furtib and others added 4 commits March 31, 2026 11:06
Co-authored-by: Kristóf Umann <dkszelethus@gmail.com>
Co-authored-by: Kristóf Umann <dkszelethus@gmail.com>
Co-authored-by: Kristóf Umann <dkszelethus@gmail.com>
@furtib furtib merged commit 3977a09 into Ericsson:main Mar 31, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test ☑️ Adding or refactoring tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants