Merge pull request #1308 from lorenzo/patch-1

Improving hadolint checker

1 files changed, 22 insertions, 1 deletions

diff --git a/test/test_dockerfile_hadolint_linter.vader b/test/test_dockerfile_hadolint_linter.vader
index 7262c5b0..3edbb2b6 100644
--- a/test/test_dockerfile_hadolint_linter.vader
+++ b/test/test_dockerfile_hadolint_linter.vader
@@ -55,7 +55,7 @@ Execute(command is correct when using docker):
   let b:ale_dockerfile_hadolint_use_docker = 'always'
 
   AssertEqual
-  \ "docker run --rm -i lukasmartinelli/hadolint",
+  \ "docker run --rm -i hadolint/hadolint",
   \ ale_linters#dockerfile#hadolint#GetCommand(bufnr(''))
 
 
@@ -66,4 +66,25 @@ Execute(command is correct when not docker):
   \ "hadolint -",
   \ ale_linters#dockerfile#hadolint#GetCommand(bufnr(''))
 
+Execute(test warnings from hadolint):
+  AssertEqual
+  \ [{'lnum': 10, 'col': 0, 'type': 'W', 'text': 'Using latest is prone to errors', 'detail': "DL3007 ( https://github.com/hadolint/hadolint/wiki/DL3007 )\n\nUsing latest is prone to errors"}],
+  \ ale_linters#dockerfile#hadolint#Handle(bufnr(''), [
+  \ '/dev/stdin:10 DL3007 Using latest is prone to errors',
+  \ ])
+
+Execute(test warnings from shellcheck):
+  AssertEqual
+  \ [{'lnum': 3, 'col': 0, 'type': 'W', 'text': 'bar is referenced but not assigned.', 'detail': "SC2154 ( https://github.com/koalaman/shellcheck/wiki/SC2154 )\n\nbar is referenced but not assigned."}],
+  \ ale_linters#dockerfile#hadolint#Handle(bufnr(''), [
+  \ '/dev/stdin:3 SC2154 bar is referenced but not assigned.',
+  \ ])
+
+Execute(test errors from dockerfile parser):
+  AssertEqual
+  \ [{'lnum': 3, 'col': 4, 'type': 'E', 'text': 'unexpected "A" expecting at least one space after ''RUN''', 'detail': 'unexpected "A" expecting at least one space after ''RUN'''}],
+  \ ale_linters#dockerfile#hadolint#Handle(bufnr(''), [
+  \ "/dev/stdin:3:4 unexpected \"A\" expecting at least one space after 'RUN'",
+  \ ])
+
 " fin...