mirror of
https://github.com/dense-analysis/ale
synced 2024-12-28 00:52:20 +00:00
aee0cc45be
* Add cppcheck handler match on misra msg * Use --file-filter cppcheck option Cppcheck recently added --file-filter so that cppcheck only checks the filtered files, even when using --project option, which checks all files in the project, by default. The --ccpcheck-build-dir option didn't help enough (at all?). * Added C test cases Also fixed and assumed typo: foo.c, instead of foo.cpp * Replace hard-coded full path filenames Attempt to fix the windows platform test execution. * Fix typo - foo.c, instead of foo.cpp * Reset buffer var between tests * Handle header files in cppcheck Cppcheck isn't designed to check header files, stand-alone. Daniel Marjamäki suggested using --suppress options to avoid FPs. * Fix Vint complaint in cppcheck handler. * Fix file path in cppcheck handler Co-authored-by: Dan George <dgeorge@anduril.com>
61 lines
2.7 KiB
Plaintext
61 lines
2.7 KiB
Plaintext
Before:
|
|
call ale#assert#SetUpLinterTest('c', 'cppcheck')
|
|
let b:command_tail = ' -q --language=c --template=' . ale#Escape('{file}:{line}:{column}: {severity}:{inconclusive:inconclusive:} {message} [{id}]\\n{code}') . ' --enable=style -I' . ale#Escape(ale#path#Simplify(g:dir)) .' %t'
|
|
|
|
After:
|
|
unlet! b:command_tail
|
|
call ale#assert#TearDownLinterTest()
|
|
|
|
Execute(The executable should be configurable):
|
|
AssertLinter 'cppcheck', ale#Escape('cppcheck') . b:command_tail
|
|
|
|
let b:ale_c_cppcheck_executable = 'foobar'
|
|
|
|
AssertLinterCwd ''
|
|
AssertLinter 'foobar', ale#Escape('foobar') . b:command_tail
|
|
|
|
Execute(cppcheck for C should detect compile_commands.json files):
|
|
call ale#test#SetFilename('../test-files/cppcheck/one/foo.c')
|
|
|
|
AssertLinterCwd ale#path#Simplify(g:dir . '/../test-files/cppcheck/one')
|
|
AssertLinter 'cppcheck', ale#Escape('cppcheck')
|
|
\ . ' -q --language=c'
|
|
\ . ' --template=' . ale#Escape('{file}:{line}:{column}: {severity}:{inconclusive:inconclusive:} {message} [{id}]\\n{code}')
|
|
\ . ' --project=' . ale#Escape('compile_commands.json')
|
|
\ . ' --file-filter=' . ale#Escape('foo.c')
|
|
\ . ' --enable=style %t'
|
|
|
|
Execute(cppcheck for C should detect compile_commands.json files in build directories):
|
|
call ale#test#SetFilename('../test-files/cppcheck/with_build_dir/foo.c')
|
|
|
|
AssertLinterCwd ale#path#Simplify(g:dir . '/../test-files/cppcheck/with_build_dir')
|
|
AssertLinter 'cppcheck', ale#Escape('cppcheck')
|
|
\ . ' -q --language=c'
|
|
\ . ' --template=' . ale#Escape('{file}:{line}:{column}: {severity}:{inconclusive:inconclusive:} {message} [{id}]\\n{code}')
|
|
\ . ' --project=' . ale#Escape(ale#path#Simplify('build/compile_commands.json'))
|
|
\ . ' --file-filter=' . ale#Escape('foo.c')
|
|
\ . ' --enable=style %t'
|
|
|
|
Execute(cppcheck for C should include file dir if compile_commands.json file is not found):
|
|
call ale#test#SetFilename('../test-files/cppcheck/foo.c')
|
|
|
|
AssertLinter 'cppcheck',
|
|
\ ale#Escape('cppcheck')
|
|
\ . ' -q --language=c'
|
|
\ . ' --template=' . ale#Escape('{file}:{line}:{column}: {severity}:{inconclusive:inconclusive:} {message} [{id}]\\n{code}')
|
|
\ . ' --enable=style'
|
|
\ . ' -I' . ale#Escape(ale#path#Simplify(g:dir . '/../test-files/cppcheck'))
|
|
\ . ' %t'
|
|
|
|
Execute(cppcheck for C header should include file dir and not use compile_commands.json):
|
|
call ale#test#SetFilename('../test-files/cppcheck/one/foo.h')
|
|
|
|
AssertLinter 'cppcheck',
|
|
\ ale#Escape('cppcheck')
|
|
\ . ' -q --language=c'
|
|
\ . ' --template=' . ale#Escape('{file}:{line}:{column}: {severity}:{inconclusive:inconclusive:} {message} [{id}]\\n{code}')
|
|
\ . ' -I' . ale#Escape(ale#path#Simplify(g:dir . '/../test-files/cppcheck/one'))
|
|
\ . ' --suppress=unusedStructMember'
|
|
\ . ' --enable=style'
|
|
\ . ' %t'
|