patch 8.2.2117: some functions use any value as a string
Problem: Some functions use any value as a string.
Solution: Check that the value is a non-empty string.
diff --git a/src/testdir/test_vim9_builtin.vim b/src/testdir/test_vim9_builtin.vim
index 50b711f..927a01f 100644
--- a/src/testdir/test_vim9_builtin.vim
+++ b/src/testdir/test_vim9_builtin.vim
@@ -185,6 +185,18 @@
count('ABC ABC ABC', 'b', false)->assert_equal(0)
enddef
+def Test_executable()
+ CheckDefExecFailure(['echo executable(true)'], 'E928:')
+ CheckDefExecFailure(['echo executable(v:null)'], 'E928:')
+ CheckDefExecFailure(['echo executable("")'], 'E928:')
+enddef
+
+def Test_exepath()
+ CheckDefExecFailure(['echo exepath(true)'], 'E928:')
+ CheckDefExecFailure(['echo exepath(v:null)'], 'E928:')
+ CheckDefExecFailure(['echo exepath("")'], 'E928:')
+enddef
+
def Test_expand()
split SomeFile
expand('%', true, true)->assert_equal(['SomeFile'])
@@ -241,6 +253,39 @@
CheckDefAndScriptSuccess(lines)
enddef
+def Test_filereadable()
+ CheckDefExecFailure(['echo filereadable(true)'], 'E928:')
+ CheckDefExecFailure(['echo filereadable(v:null)'], 'E928:')
+ CheckDefExecFailure(['echo filereadable("")'], 'E928:')
+enddef
+
+def Test_filewritable()
+ CheckDefExecFailure(['echo filewritable(true)'], 'E928:')
+ CheckDefExecFailure(['echo filewritable(v:null)'], 'E928:')
+ CheckDefExecFailure(['echo filewritable("")'], 'E928:')
+enddef
+
+def Test_finddir()
+ CheckDefExecFailure(['echo finddir(true)'], 'E928:')
+ CheckDefExecFailure(['echo finddir(v:null)'], 'E928:')
+ CheckDefExecFailure(['echo finddir("")'], 'E928:')
+enddef
+
+def Test_findfile()
+ CheckDefExecFailure(['echo findfile(true)'], 'E928:')
+ CheckDefExecFailure(['echo findfile(v:null)'], 'E928:')
+ CheckDefExecFailure(['echo findfile("")'], 'E928:')
+enddef
+
+def Test_fnamemodify()
+ CheckDefExecFailure(['echo fnamemodify(true, ":p")'], 'E928:')
+ CheckDefExecFailure(['echo fnamemodify(v:null, ":p")'], 'E928:')
+ CheckDefExecFailure(['echo fnamemodify("", ":p")'], 'E928:')
+ CheckDefExecFailure(['echo fnamemodify("file", true)'], 'E928:')
+ CheckDefExecFailure(['echo fnamemodify("file", v:null)'], 'E928:')
+ CheckDefExecFailure(['echo fnamemodify("file", "")'], 'E928:')
+enddef
+
def Test_filter_wrong_dict_key_type()
assert_fails('Wrong_dict_key_type([1, 2, 3])', 'E1012:')
enddef
@@ -313,6 +358,30 @@
d->assert_equal({items: []})
enddef
+def Test_getfperm()
+ CheckDefExecFailure(['echo getfperm(true)'], 'E928:')
+ CheckDefExecFailure(['echo getfperm(v:null)'], 'E928:')
+ CheckDefExecFailure(['echo getfperm("")'], 'E928:')
+enddef
+
+def Test_getfsize()
+ CheckDefExecFailure(['echo getfsize(true)'], 'E928:')
+ CheckDefExecFailure(['echo getfsize(v:null)'], 'E928:')
+ CheckDefExecFailure(['echo getfsize("")'], 'E928:')
+enddef
+
+def Test_getftime()
+ CheckDefExecFailure(['echo getftime(true)'], 'E928:')
+ CheckDefExecFailure(['echo getftime(v:null)'], 'E928:')
+ CheckDefExecFailure(['echo getftime("")'], 'E928:')
+enddef
+
+def Test_getftype()
+ CheckDefExecFailure(['echo getftype(true)'], 'E928:')
+ CheckDefExecFailure(['echo getftype(v:null)'], 'E928:')
+ CheckDefExecFailure(['echo getftype("")'], 'E928:')
+enddef
+
def Test_getqflist_return_type()
var l = getqflist()
l->assert_equal([])
diff --git a/src/testdir/test_vim9_expr.vim b/src/testdir/test_vim9_expr.vim
index 10230d5..812aabf 100644
--- a/src/testdir/test_vim9_expr.vim
+++ b/src/testdir/test_vim9_expr.vim
@@ -1971,7 +1971,7 @@
CheckDefExecFailure(['var x: dict<string> = {a: "x", b: 134}'], 'E1012:', 1)
CheckDefFailure(['var x = ({'], 'E723:', 2)
- CheckDefExecFailure(['{}[getftype("")]'], 'E716: Key not present in Dictionary: ""', 1)
+ CheckDefExecFailure(['{}[getftype("file")]'], 'E716: Key not present in Dictionary: ""', 1)
enddef
def Test_expr7_dict_vim9script()