diff options
author | Yegappan Lakshmanan <yegappan@yahoo.com> | 2022-09-22 13:44:00 +0100 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2022-09-22 13:44:00 +0100 |
commit | e24b5e0b0f5ab015215ef2761baa98ccb1ba8606 (patch) | |
tree | efd113e4e4e40d32cf12fa64b50e3dd29f05c00a | |
parent | f1d2ddcb9b298d8ed7d31c792c56531211fbd771 (diff) |
patch 9.0.0543: insufficient testing for assert and test functionsv9.0.0543
Problem: Insufficient testing for assert and test functions.
Solution: Add a few more tests. (Yegappan Lakshmanan, closes #11190)
-rw-r--r-- | src/testdir/test_assert.vim | 52 | ||||
-rw-r--r-- | src/testdir/test_options.vim | 1 | ||||
-rw-r--r-- | src/testdir/test_vimscript.vim | 8 | ||||
-rw-r--r-- | src/version.c | 2 |
4 files changed, 63 insertions, 0 deletions
diff --git a/src/testdir/test_assert.vim b/src/testdir/test_assert.vim index b666c011e3..89ac012840 100644 --- a/src/testdir/test_assert.vim +++ b/src/testdir/test_assert.vim @@ -269,6 +269,20 @@ func Test_assert_fail_fails() call assert_match("E856: \"assert_fails()\" second argument", exp) try + call assert_equal(1, assert_fails('xxx', test_null_list())) + catch + let exp = v:exception + endtry + call assert_match("E856: \"assert_fails()\" second argument", exp) + + try + call assert_equal(1, assert_fails('xxx', [])) + catch + let exp = v:exception + endtry + call assert_match("E856: \"assert_fails()\" second argument", exp) + + try call assert_equal(1, assert_fails('xxx', #{one: 1})) catch let exp = v:exception @@ -307,6 +321,15 @@ func Test_assert_fail_fails() call assert_equal(1, assert_fails('c0', ['', '\(.\)\1'])) call assert_match("Expected '\\\\\\\\(.\\\\\\\\)\\\\\\\\1' but got 'E939: Positive count required: c0': c0", v:errors[0]) call remove(v:errors, 0) + + " Test for matching the line number and the script name in an error message + call writefile(['', 'call Xnonexisting()'], 'Xassertfails.vim', 'D') + call assert_fails('source Xassertfails.vim', 'E117:', '', 10) + call assert_match("Expected 10 but got 2", v:errors[0]) + call remove(v:errors, 0) + call assert_fails('source Xassertfails.vim', 'E117:', '', 2, 'Xabc') + call assert_match("Expected 'Xabc' but got .*Xassertfails.vim", v:errors[0]) + call remove(v:errors, 0) endfunc func Test_assert_fails_in_try_block() @@ -331,6 +354,12 @@ func Test_assert_beeps() bwipe endfunc +func Test_assert_nobeep() + call assert_equal(1, assert_nobeep('normal! cr')) + call assert_match("command did beep: normal! cr", v:errors[0]) + call remove(v:errors, 0) +endfunc + func Test_assert_inrange() call assert_equal(0, assert_inrange(7, 7, 7)) call assert_equal(0, assert_inrange(5, 7, 5)) @@ -365,6 +394,16 @@ func Test_assert_inrange() call assert_equal(1, assert_inrange(5, 7, 8.0)) call assert_match("Expected range 5.0 - 7.0, but got 8.0", v:errors[0]) call remove(v:errors, 0) + + " Use a custom message + call assert_equal(1, assert_inrange(5, 7, 8.0, "Higher")) + call assert_match("Higher", v:errors[0]) + call remove(v:errors, 0) + + " Invalid arguments + call assert_fails("call assert_inrange([], 2, 3)", 'E1219:') + call assert_fails("call assert_inrange(1, [], 3)", 'E1219:') + call assert_fails("call assert_inrange(1, 2, [])", 'E1219:') endfunc func Test_assert_with_msg() @@ -407,6 +446,19 @@ func Test_user_is_happy() sleep 300m endfunc +" Test for the test_alloc_fail() function +func Test_test_alloc_fail() + call assert_fails('call test_alloc_fail([], 1, 1)', 'E474:') + call assert_fails('call test_alloc_fail(10, [], 1)', 'E474:') + call assert_fails('call test_alloc_fail(10, 1, [])', 'E474:') + call assert_fails('call test_alloc_fail(999999, 1, 1)', 'E474:') +endfunc + +" Test for the test_option_not_set() function +func Test_test_option_not_set() + call assert_fails('call test_option_not_set("Xinvalidopt")', 'E475:') +endfunc + " Must be last. func Test_zz_quit_detected() " Verify that if a test function ends Vim the test script detects this. diff --git a/src/testdir/test_options.vim b/src/testdir/test_options.vim index ec114942a7..a0ee53bb30 100644 --- a/src/testdir/test_options.vim +++ b/src/testdir/test_options.vim @@ -815,6 +815,7 @@ func Test_shortmess_F2() call assert_match('file2', execute('bn', '')) bwipe bwipe + call assert_fails('call test_getvalue("abc")', 'E475:') endfunc func Test_local_scrolloff() diff --git a/src/testdir/test_vimscript.vim b/src/testdir/test_vimscript.vim index 641ef73687..affb958f1d 100644 --- a/src/testdir/test_vimscript.vim +++ b/src/testdir/test_vimscript.vim @@ -7216,6 +7216,14 @@ func Test_refcount() call d.Func() unlet d delfunc DictFunc + + call assert_equal(-1, test_refcount(test_null_job())) + call assert_equal(-1, test_refcount(test_null_channel())) + call assert_equal(-1, test_refcount(test_null_function())) + call assert_equal(-1, test_refcount(test_null_partial())) + call assert_equal(-1, test_refcount(test_null_blob())) + call assert_equal(-1, test_refcount(test_null_list())) + call assert_equal(-1, test_refcount(test_null_dict())) endfunc " Test for missing :endif, :endfor, :endwhile and :endtry {{{1 diff --git a/src/version.c b/src/version.c index d1ed0c4eae..d3694560f7 100644 --- a/src/version.c +++ b/src/version.c @@ -700,6 +700,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 543, +/**/ 542, /**/ 541, |