2017-05-07 11:16:17 -04:00
|
|
|
Before:
|
2018-07-15 13:24:53 -04:00
|
|
|
call ale#assert#SetUpLinterTest('python', 'flake8')
|
2017-10-08 18:26:50 -04:00
|
|
|
let b:bin_dir = has('win32') ? 'Scripts' : 'bin'
|
|
|
|
|
2018-07-15 13:24:53 -04:00
|
|
|
WithChainResults ['3.0.0']
|
2017-05-07 11:16:17 -04:00
|
|
|
|
|
|
|
After:
|
2017-10-08 18:26:50 -04:00
|
|
|
unlet! b:executable
|
2018-07-15 13:24:53 -04:00
|
|
|
unlet! b:bin_dir
|
|
|
|
call ale#assert#TearDownLinterTest()
|
2017-05-07 11:16:17 -04:00
|
|
|
|
|
|
|
Execute(The flake8 callbacks should return the correct default values):
|
2018-07-15 13:24:53 -04:00
|
|
|
AssertLinter 'flake8', [
|
2017-10-08 18:26:50 -04:00
|
|
|
\ ale#Escape('flake8') . ' --version',
|
2018-04-26 19:53:44 -04:00
|
|
|
\ ale#path#BufferCdString(bufnr(''))
|
|
|
|
\ . ale#Escape('flake8') . ' --format=default --stdin-display-name %s -',
|
2018-07-15 13:24:53 -04:00
|
|
|
\]
|
|
|
|
|
|
|
|
" The version check should be cached.
|
|
|
|
WithChainResults []
|
|
|
|
AssertLinter 'flake8', [
|
|
|
|
\ '',
|
|
|
|
\ ale#path#BufferCdString(bufnr(''))
|
|
|
|
\ . ale#Escape('flake8') . ' --format=default --stdin-display-name %s -',
|
|
|
|
\]
|
2017-11-09 18:42:54 -05:00
|
|
|
|
2017-05-07 11:16:17 -04:00
|
|
|
" Try with older versions.
|
2017-11-09 18:42:54 -05:00
|
|
|
call ale#semver#ResetVersionCache()
|
2018-07-15 13:24:53 -04:00
|
|
|
WithChainResults ['2.9.9']
|
|
|
|
AssertLinter 'flake8', [
|
|
|
|
\ ale#Escape('flake8') . ' --version',
|
2018-04-26 19:53:44 -04:00
|
|
|
\ ale#path#BufferCdString(bufnr(''))
|
|
|
|
\ . ale#Escape('flake8') . ' --format=default -',
|
2018-07-15 13:24:53 -04:00
|
|
|
\]
|
2017-05-07 11:16:17 -04:00
|
|
|
|
2018-05-28 11:34:54 -04:00
|
|
|
Execute(The option for disabling changing directories should work):
|
|
|
|
let g:ale_python_flake8_change_directory = 0
|
|
|
|
|
2018-07-15 13:24:53 -04:00
|
|
|
AssertLinter 'flake8', [
|
|
|
|
\ ale#Escape('flake8') . ' --version',
|
2018-05-28 11:34:54 -04:00
|
|
|
\ ale#Escape('flake8') . ' --format=default --stdin-display-name %s -',
|
2018-07-15 13:24:53 -04:00
|
|
|
\]
|
2018-05-28 11:34:54 -04:00
|
|
|
|
2017-05-07 11:16:17 -04:00
|
|
|
Execute(The flake8 command callback should let you set options):
|
|
|
|
let g:ale_python_flake8_options = '--some-option'
|
|
|
|
|
2018-07-15 13:24:53 -04:00
|
|
|
WithChainResults ['3.0.4']
|
|
|
|
AssertLinter 'flake8', [
|
|
|
|
\ ale#Escape('flake8') . ' --version',
|
2018-04-26 19:53:44 -04:00
|
|
|
\ ale#path#BufferCdString(bufnr(''))
|
2018-07-15 13:24:53 -04:00
|
|
|
\ . ale#Escape('flake8') . ' --some-option'
|
|
|
|
\ . ' --format=default --stdin-display-name %s -',
|
|
|
|
\]
|
2017-11-09 18:42:54 -05:00
|
|
|
|
|
|
|
call ale#semver#ResetVersionCache()
|
2018-07-15 13:24:53 -04:00
|
|
|
WithChainResults ['2.9.9']
|
|
|
|
AssertLinter 'flake8', [
|
|
|
|
\ ale#Escape('flake8') . ' --version',
|
2018-04-26 19:53:44 -04:00
|
|
|
\ ale#path#BufferCdString(bufnr(''))
|
2018-07-15 13:24:53 -04:00
|
|
|
\ . ale#Escape('flake8') . ' --some-option --format=default -',
|
|
|
|
\]
|
2017-05-07 11:16:17 -04:00
|
|
|
|
|
|
|
Execute(You should be able to set a custom executable and it should be escaped):
|
|
|
|
let g:ale_python_flake8_executable = 'executable with spaces'
|
|
|
|
|
2018-07-15 13:24:53 -04:00
|
|
|
AssertLinter 'executable with spaces', [
|
2017-10-08 18:26:50 -04:00
|
|
|
\ ale#Escape('executable with spaces') . ' --version',
|
2018-04-26 19:53:44 -04:00
|
|
|
\ ale#path#BufferCdString(bufnr(''))
|
|
|
|
\ . ale#Escape('executable with spaces')
|
2017-10-08 18:26:50 -04:00
|
|
|
\ . ' --format=default'
|
|
|
|
\ . ' --stdin-display-name %s -',
|
2018-07-15 13:24:53 -04:00
|
|
|
\]
|
2017-05-07 11:16:17 -04:00
|
|
|
|
|
|
|
Execute(The flake8 callbacks should detect virtualenv directories):
|
|
|
|
silent execute 'file ' . fnameescape(g:dir . '/python_paths/with_virtualenv/subdir/foo/bar.py')
|
|
|
|
|
2017-12-19 13:23:09 -05:00
|
|
|
let b:executable = ale#path#Simplify(
|
2017-10-08 18:26:50 -04:00
|
|
|
\ g:dir . '/python_paths/with_virtualenv/env/' . b:bin_dir . '/flake8'
|
|
|
|
\)
|
|
|
|
|
2018-07-15 13:24:53 -04:00
|
|
|
AssertLinter b:executable, [
|
2017-10-08 18:26:50 -04:00
|
|
|
\ ale#Escape(b:executable) . ' --version',
|
2018-04-26 19:53:44 -04:00
|
|
|
\ ale#path#BufferCdString(bufnr(''))
|
2018-07-15 13:24:53 -04:00
|
|
|
\ . ale#Escape(b:executable)
|
|
|
|
\ . ' --format=default'
|
|
|
|
\ . ' --stdin-display-name %s -',
|
|
|
|
\]
|
2017-05-07 11:31:33 -04:00
|
|
|
|
2017-07-11 16:57:37 -04:00
|
|
|
Execute(The FindProjectRoot should detect the project root directory for namespace package via Manifest.in):
|
|
|
|
silent execute 'file ' . fnameescape(g:dir . '/python_paths/namespace_package_manifest/namespace/foo/bar.py')
|
|
|
|
|
|
|
|
AssertEqual
|
2017-12-19 13:23:09 -05:00
|
|
|
\ ale#path#Simplify(g:dir . '/python_paths/namespace_package_manifest'),
|
2017-07-11 16:57:37 -04:00
|
|
|
\ ale#python#FindProjectRoot(bufnr(''))
|
|
|
|
|
|
|
|
Execute(The FindProjectRoot should detect the project root directory for namespace package via setup.cf):
|
|
|
|
silent execute 'file ' . fnameescape(g:dir . '/python_paths/namespace_package_setup/namespace/foo/bar.py')
|
|
|
|
|
|
|
|
AssertEqual
|
2017-12-19 13:23:09 -05:00
|
|
|
\ ale#path#Simplify(g:dir . '/python_paths/namespace_package_setup'),
|
2017-07-11 16:57:37 -04:00
|
|
|
\ ale#python#FindProjectRoot(bufnr(''))
|
|
|
|
|
|
|
|
Execute(The FindProjectRoot should detect the project root directory for namespace package via pytest.ini):
|
|
|
|
silent execute 'file ' . fnameescape(g:dir . '/python_paths/namespace_package_pytest/namespace/foo/bar.py')
|
|
|
|
|
|
|
|
AssertEqual
|
2017-12-19 13:23:09 -05:00
|
|
|
\ ale#path#Simplify(g:dir . '/python_paths/namespace_package_pytest'),
|
2017-07-11 16:57:37 -04:00
|
|
|
\ ale#python#FindProjectRoot(bufnr(''))
|
|
|
|
|
|
|
|
Execute(The FindProjectRoot should detect the project root directory for namespace package via tox.ini):
|
|
|
|
silent execute 'file ' . fnameescape(g:dir . '/python_paths/namespace_package_tox/namespace/foo/bar.py')
|
|
|
|
|
|
|
|
AssertEqual
|
2017-12-19 13:23:09 -05:00
|
|
|
\ ale#path#Simplify(g:dir . '/python_paths/namespace_package_tox'),
|
2017-07-11 16:57:37 -04:00
|
|
|
\ ale#python#FindProjectRoot(bufnr(''))
|
|
|
|
|
|
|
|
Execute(The FindProjectRoot should detect the project root directory for non-namespace package):
|
|
|
|
silent execute 'file ' . fnameescape(g:dir . '/python_paths/no_virtualenv/subdir/foo/bar.py')
|
|
|
|
|
|
|
|
AssertEqual
|
2017-12-19 13:23:09 -05:00
|
|
|
\ ale#path#Simplify(g:dir . '/python_paths/no_virtualenv/subdir'),
|
2017-07-11 16:57:37 -04:00
|
|
|
\ ale#python#FindProjectRoot(bufnr(''))
|
|
|
|
|
2017-05-07 11:31:33 -04:00
|
|
|
" Some users currently run flake8 this way, so we should support it.
|
|
|
|
Execute(Using `python -m flake8` should be supported for running flake8):
|
|
|
|
silent execute 'file ' . fnameescape(g:dir . '/python_paths/with_virtualenv/subdir/foo/bar.py')
|
|
|
|
|
|
|
|
let g:ale_python_flake8_executable = 'python'
|
|
|
|
let g:ale_python_flake8_options = '-m flake8 --some-option'
|
|
|
|
|
2018-07-15 13:24:53 -04:00
|
|
|
WithChainResults ['2.9.9']
|
|
|
|
AssertLinter 'python', [
|
2017-10-08 18:26:50 -04:00
|
|
|
\ ale#Escape('python') . ' -m flake8 --version',
|
2018-04-26 19:53:44 -04:00
|
|
|
\ ale#path#BufferCdString(bufnr(''))
|
2018-07-15 13:24:53 -04:00
|
|
|
\ . ale#Escape('python')
|
|
|
|
\ . ' -m flake8 --some-option --format=default -'
|
|
|
|
\]
|
2017-05-07 11:31:33 -04:00
|
|
|
|
2017-11-09 18:42:54 -05:00
|
|
|
call ale#semver#ResetVersionCache()
|
2017-05-07 11:31:33 -04:00
|
|
|
|
|
|
|
" Leading spaces shouldn't matter
|
|
|
|
let g:ale_python_flake8_options = ' -m flake8 --some-option'
|
|
|
|
|
2018-07-15 13:24:53 -04:00
|
|
|
WithChainResults ['2.9.9']
|
|
|
|
AssertLinter 'python', [
|
2017-10-08 18:26:50 -04:00
|
|
|
\ ale#Escape('python') . ' -m flake8 --version',
|
2018-04-26 19:53:44 -04:00
|
|
|
\ ale#path#BufferCdString(bufnr(''))
|
2018-07-15 13:24:53 -04:00
|
|
|
\ . ale#Escape('python')
|
|
|
|
\ . ' -m flake8 --some-option --format=default -'
|
|
|
|
\]
|
2018-06-02 21:49:12 -04:00
|
|
|
|
2018-07-15 13:24:53 -04:00
|
|
|
Execute(Setting executable to 'pipenv' should append 'run flake8'):
|
2018-06-02 21:49:12 -04:00
|
|
|
let g:ale_python_flake8_executable = 'path/to/pipenv'
|
|
|
|
|
2018-07-15 13:24:53 -04:00
|
|
|
" FIXME: pipenv should check the vresion with flake8.
|
|
|
|
WithChainResults []
|
|
|
|
AssertLinter 'path/to/pipenv',
|
2018-06-02 21:49:12 -04:00
|
|
|
\ ale#path#BufferCdString(bufnr(''))
|
2018-07-15 13:24:53 -04:00
|
|
|
\ . ale#Escape('path/to/pipenv') . ' run flake8 --format=default -'
|