Skip to content

test/cfg: enabled information and --inconclusive in all cases / some cleanups#4784

Merged
danmar merged 7 commits into
cppcheck-opensource:mainfrom
firewave:std-test
Feb 24, 2023
Merged

test/cfg: enabled information and --inconclusive in all cases / some cleanups#4784
danmar merged 7 commits into
cppcheck-opensource:mainfrom
firewave:std-test

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

@firewave firewave commented Feb 9, 2023

No description provided.

@firewave
Copy link
Copy Markdown
Collaborator Author

firewave commented Feb 9, 2023

Luckily there were no hidden false negatives.

Turns out the tests were correctly configured in CMake.

@firewave firewave changed the title test/cfg/runtests.sh: enabled information and --inconclusive / some cleanups test/cfg: enabled information and --inconclusive in all cases / some cleanups Feb 10, 2023
@firewave
Copy link
Copy Markdown
Collaborator Author

posix.cfg and gnu.cfg both contain the mmap() function leading to different tests results depending on the order of the libraries.

I am not sure how this could be handled properly. It is properly correct but the gnu test should not depend on posix and there should possibly some indication that data was changed as it might not be intentional.

This is another reason to only include libraries if necessary - see https://trac.cppcheck.net/ticket/11287.

@danmar danmar merged commit da09a92 into cppcheck-opensource:main Feb 24, 2023
@firewave firewave deleted the std-test branch February 24, 2023 20:54
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