From 92e6e4d1ba482a4d2d89d850f660c67ccf8a28eb Mon Sep 17 00:00:00 2001
From: Anthony DeDominic <anthony@dedominic.pw>
Date: Wed, 14 Mar 2018 13:46:57 -0400
Subject: Fix awk linter and security concerns. (#1411)

* Fixed (g)awk linter

 * Made it secure, albeit less useful.
 * Added gawk handler; the cpplint one was not working?
 * Added gawk handler test.
 * added warning to gawk handler.
 * added gawk command callback test
 * added comment about --source

* added back optional commandline option
---
 .../test_gawk_command_callback.vader               | 40 ++++++++++++++++++++++
 test/handler/test_gawk_handler.vader               | 39 +++++++++++++++++++++
 2 files changed, 79 insertions(+)
 create mode 100644 test/command_callback/test_gawk_command_callback.vader
 create mode 100644 test/handler/test_gawk_handler.vader

(limited to 'test')

diff --git a/test/command_callback/test_gawk_command_callback.vader b/test/command_callback/test_gawk_command_callback.vader
new file mode 100644
index 00000000..ae128fe5
--- /dev/null
+++ b/test/command_callback/test_gawk_command_callback.vader
@@ -0,0 +1,40 @@
+Before:
+  Save g:ale_awk_gawk_executable
+  Save g:ale_awk_gawk_options
+  unlet! g:ale_awk_gawk_executable
+  unlet! g:ale_awk_gawk_options
+
+  runtime ale_linters/awk/gawk.vim
+
+After:
+  Restore
+  unlet! b:command_tail
+  unlet! b:ale_awk_gawk_executable
+  unlet! b:ale_awk_gawk_options
+
+  call ale#linter#Reset()
+
+Execute(The executable should be used in the command):
+  AssertEqual
+  \ 'gawk'
+    \ . " --source 'BEGIN { exit } END { exit 1 }'"
+    \ . '  ' . '-f %t --lint /dev/null',
+  \ ale_linters#awk#gawk#GetCommand(bufnr(''))
+
+  let b:ale_awk_gawk_executable = '/other/gawk'
+
+  AssertEqual
+  \ '/other/gawk'
+    \ . " --source 'BEGIN { exit } END { exit 1 }'"
+    \ . '  ' . '-f %t --lint /dev/null',
+  \ ale_linters#awk#gawk#GetCommand(bufnr(''))
+
+  let b:ale_awk_gawk_executable = 'gawk'
+  let b:ale_awk_gawk_options = '--something'
+
+  AssertEqual
+  \ 'gawk'
+    \ . " --source 'BEGIN { exit } END { exit 1 }'"
+    \ . ' --something'
+    \ . ' ' . '-f %t --lint /dev/null',
+  \ ale_linters#awk#gawk#GetCommand(bufnr(''))
diff --git a/test/handler/test_gawk_handler.vader b/test/handler/test_gawk_handler.vader
new file mode 100644
index 00000000..3a7b5457
--- /dev/null
+++ b/test/handler/test_gawk_handler.vader
@@ -0,0 +1,39 @@
+Before:
+  runtime ale_linters/awk/gawk.vim
+
+After:
+  call ale#linter#Reset()
+
+Execute(gawk syntax errors should be parsed correctly):
+
+  AssertEqual
+  \ [
+  \   {
+  \     'lnum': 1,
+  \     'col': 0,
+  \     'text': "invalid char ''' in expression",
+  \     'code': 0,
+  \     'type': 'E',
+  \   },
+  \   {
+  \     'lnum': 5,
+  \     'col': 0,
+  \     'text': 'unterminated string',
+  \     'code': 0,
+  \     'type': 'E',
+  \   },
+  \   {
+  \     'lnum': 10,
+  \     'col': 0,
+  \     'text': "escape sequence `\u' treated as plain `u'",
+  \     'code': 0,
+  \     'type': 'W',
+  \   },
+  \ ],
+  \ ale#handlers#gawk#HandleGawkFormat(347, [
+  \   "gawk: something.awk:1: BEGIN { system('touch aaaaaaaaa') }",
+  \   "gawk: something.awk:1:                ^ invalid char ''' in expression",
+  \   'gawk: something.awk:5: { x = "aaaaaaaaaaa',
+  \   'gawk: something.awk:5:       ^ unterminated string',
+  \   "gawk: something.awk:10: warning: escape sequence `\u' treated as plain `u'",
+  \ ])
-- 
cgit v1.2.3