3c799abb44
The existing option setting handles setting additional compile flags to pass to clang-tidy. The new option setting added here allows setting additional clang-tidy specific flags to be passed as well. Fixes #2324
79 lines
2.5 KiB
Plaintext
79 lines
2.5 KiB
Plaintext
Before:
|
|
call ale#assert#SetUpLinterTest('cpp', 'clangtidy')
|
|
call ale#test#SetFilename('test.cpp')
|
|
|
|
After:
|
|
call ale#assert#TearDownLinterTest()
|
|
|
|
Execute(The clangtidy command default should be correct):
|
|
AssertLinter 'clang-tidy',
|
|
\ ale#Escape('clang-tidy') . ' %s'
|
|
|
|
Execute(You should be able to remove the -checks option for clang-tidy):
|
|
let b:ale_cpp_clangtidy_checks = []
|
|
|
|
AssertLinter 'clang-tidy',
|
|
\ ale#Escape('clang-tidy') . ' %s'
|
|
|
|
Execute(You should be able to set other checks for clang-tidy):
|
|
let b:ale_cpp_clangtidy_checks = ['-*', 'clang-analyzer-*']
|
|
|
|
AssertLinter 'clang-tidy',
|
|
\ ale#Escape('clang-tidy')
|
|
\ . ' -checks=' . ale#Escape('-*,clang-analyzer-*') . ' %s'
|
|
|
|
Execute(You should be able to manually set compiler flags for clang-tidy):
|
|
let b:ale_cpp_clangtidy_checks = ['*']
|
|
let b:ale_cpp_clangtidy_options = '-Wall'
|
|
|
|
AssertLinter 'clang-tidy',
|
|
\ ale#Escape('clang-tidy') . ' -checks=' . ale#Escape('*') . ' %s -- -Wall'
|
|
|
|
Execute(You should be able to manually set flags for clang-tidy):
|
|
let b:ale_cpp_clangtidy_extra_options = '-config='
|
|
|
|
AssertLinter 'clang-tidy',
|
|
\ ale#Escape('clang-tidy') . ' ' . ale#Escape('-config=') . ' %s'
|
|
|
|
Execute(The build directory should be configurable):
|
|
let b:ale_cpp_clangtidy_checks = ['*']
|
|
let b:ale_c_build_dir = '/foo/bar'
|
|
|
|
AssertLinter 'clang-tidy',
|
|
\ ale#Escape('clang-tidy')
|
|
\ . ' -checks=' . ale#Escape('*') . ' %s'
|
|
\ . ' -p ' . ale#Escape('/foo/bar')
|
|
|
|
Execute(The build directory setting should override the options):
|
|
let b:ale_cpp_clangtidy_checks = ['*']
|
|
let b:ale_c_build_dir = '/foo/bar'
|
|
let b:ale_cpp_clangtidy_options = '-Wall'
|
|
|
|
AssertLinter 'clang-tidy',
|
|
\ ale#Escape('clang-tidy')
|
|
\ . ' -checks=' . ale#Escape('*') . ' %s'
|
|
\ . ' -p ' . ale#Escape('/foo/bar')
|
|
|
|
Execute(The build directory should be ignored for header files):
|
|
call ale#test#SetFilename('test.h')
|
|
|
|
let b:ale_cpp_clangtidy_checks = ['*']
|
|
let b:ale_c_build_dir = '/foo/bar'
|
|
let b:ale_cpp_clangtidy_options = '-Wall'
|
|
|
|
AssertLinter 'clang-tidy',
|
|
\ ale#Escape('clang-tidy')
|
|
\ . ' -checks=' . ale#Escape('*') . ' %s -- -Wall'
|
|
|
|
call ale#test#SetFilename('test.hpp')
|
|
|
|
AssertLinter 'clang-tidy',
|
|
\ ale#Escape('clang-tidy') . ' -checks=' . ale#Escape('*') . ' %s -- -Wall'
|
|
|
|
Execute(The executable should be configurable):
|
|
let b:ale_cpp_clangtidy_checks = ['*']
|
|
let b:ale_cpp_clangtidy_executable = 'foobar'
|
|
|
|
AssertLinter 'foobar',
|
|
\ ale#Escape('foobar') . ' -checks=' . ale#Escape('*') . ' %s'
|