summaryrefslogtreecommitdiff
path: root/test/command_callback/test_bandit_command_callback.vader
diff options
context:
space:
mode:
authorKevin Locke <kevin@kevinlocke.name>2019-02-17 14:11:56 -0700
committerKevin Locke <kevin@kevinlocke.name>2019-02-17 14:24:18 -0700
commit52c24007860b1e29c2e2cb318f29aff9889cb3c0 (patch)
treefc24fd3dc06eaad3d613619eef1efe6c9b7009e9 /test/command_callback/test_bandit_command_callback.vader
parentc3d4e0983b4b08e61692cdd88990a79525a78707 (diff)
downloadale-52c24007860b1e29c2e2cb318f29aff9889cb3c0.zip
python/bandit: Use .bandit configuration file
Bandit automatically [uses any .bandit file] within the directories on which it is invoked. Since ALE invokes bandit on stdin, it does not load a .bandit file automatically. Add support for automatically finding a .bandit file and passing it to bandit via the --ini option along with a variable to disable this behavior if desired. Note: This is useful for the skips and tests configuration options, but not exclude which would require invoking bandit using a file name, which may or may not be a good trade-off. [uses any .bandit file]: https://github.com/PyCQA/bandit/blob/1.5.1/bandit/cli/main.py#L70-L73 Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
Diffstat (limited to 'test/command_callback/test_bandit_command_callback.vader')
-rw-r--r--test/command_callback/test_bandit_command_callback.vader22
1 files changed, 22 insertions, 0 deletions
diff --git a/test/command_callback/test_bandit_command_callback.vader b/test/command_callback/test_bandit_command_callback.vader
index 5d1e6fd3..274ce901 100644
--- a/test/command_callback/test_bandit_command_callback.vader
+++ b/test/command_callback/test_bandit_command_callback.vader
@@ -47,3 +47,25 @@ Execute(Pipenv is detected when python_bandit_auto_pipenv is set):
\ . ' run bandit'
\ . b:bandit_flags
\ . ' -'
+
+Execute(The bandit command callback should add .bandit by default):
+ silent execute 'file ' . fnameescape(g:dir . '/python_paths/with_bandit/namespace/foo/bar.py')
+
+ let b:config_path = ale#path#Simplify(
+ \ g:dir . '/python_paths/with_bandit/.bandit'
+ \)
+
+ AssertLinter 'bandit',
+ \ ale#Escape('bandit')
+ \ . ' --ini ' . ale#Escape(b:config_path)
+ \ . b:bandit_flags
+ \ . ' -'
+
+Execute(The bandit command callback should support not using .bandit):
+ silent execute 'file ' . fnameescape(g:dir . '/python_paths/with_bandit/subdir/foo/bar.py')
+ let g:ale_python_bandit_use_config = 0
+
+ AssertLinter 'bandit',
+ \ ale#Escape('bandit')
+ \ . b:bandit_flags
+ \ . ' -'