]> granicus.if.org Git - vim/commitdiff
patch 8.2.3715: Vim9: valgrind reports spurious problems for a test v8.2.3715
authorBram Moolenaar <Bram@vim.org>
Wed, 1 Dec 2021 12:41:31 +0000 (12:41 +0000)
committerBram Moolenaar <Bram@vim.org>
Wed, 1 Dec 2021 12:41:31 +0000 (12:41 +0000)
Problem:    Vim9: valgrind reports spurious problems for a test.
Solution:   Move the test to the set that is known to fail.

src/testdir/test_vim9_builtin.vim
src/testdir/test_vim9_fails.vim
src/version.c

index a0acac5fe4a4852284cf4b22a96cc1491b681d5b..701caa666973315012dd05de34b4bf5947517015 100644 (file)
@@ -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')
index e09eb6543d44f1f69eb54f0b17155af3499885f0..216ef0b286f853073ec1f152df9d6ae43576eda2 100644 (file)
@@ -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
+
index 080c687edc0fff1e5f51a3528d03b81cf4c98ff9..b98d0b3ca9ab4c6347d61727e93c5c338cbabcf4 100644 (file)
@@ -753,6 +753,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    3715,
 /**/
     3714,
 /**/