changeset 26368:744fdb15347d v8.2.3715

patch 8.2.3715: Vim9: valgrind reports spurious problems for a test Commit: https://github.com/vim/vim/commit/f0e496a85a89fb1fd21b6af6363dcfc276331110 Author: Bram Moolenaar <Bram@vim.org> Date: Wed Dec 1 12:41:31 2021 +0000 patch 8.2.3715: Vim9: valgrind reports spurious problems for a test Problem: Vim9: valgrind reports spurious problems for a test. Solution: Move the test to the set that is known to fail.
author Bram Moolenaar <Bram@vim.org>
date Wed, 01 Dec 2021 13:45:03 +0100
parents d625a0a8740f
children da56c576e583
files src/testdir/test_vim9_builtin.vim src/testdir/test_vim9_fails.vim src/version.c
diffstat 3 files changed, 17 insertions(+), 11 deletions(-) [+]
line wrap: on
line diff
--- a/src/testdir/test_vim9_builtin.vim
+++ b/src/testdir/test_vim9_builtin.vim
@@ -1879,17 +1879,7 @@ def Test_job_info()
   endif
 enddef
 
-def Test_job_info_return_type()
-  if !has('job')
-    CheckFeature job
-  else
-    job_start(&shell)
-    var jobs = job_info()
-    assert_equal('list<job>', typename(jobs))
-    assert_equal('dict<any>', typename(job_info(jobs[0])))
-    job_stop(jobs[0])
-  endif
-enddef
+" Test_job_info_return_type() is in test_vim9_fails.vim
 
 def Test_job_setoptions()
   if !has('job')
--- a/src/testdir/test_vim9_fails.vim
+++ b/src/testdir/test_vim9_fails.vim
@@ -8,3 +8,17 @@ def Test_assignment()
     var job2: job = job_start('willfail')
   endif
 enddef
+
+" Unclear why this test causes valgrind to report problems.
+def Test_job_info_return_type()
+  if !has('job')
+    CheckFeature job
+  else
+    var job: job = job_start(&shell)
+    var jobs = job_info()
+    assert_equal('list<job>', typename(jobs))
+    assert_equal('dict<any>', typename(job_info(jobs[0])))
+    job_stop(job)
+  endif
+enddef
+
--- a/src/version.c
+++ b/src/version.c
@@ -754,6 +754,8 @@ static char *(features[]) =
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    3715,
+/**/
     3714,
 /**/
     3713,