Merge pull request #5336 from bfredl/del_var

 make del_var and set_var not return the old value.

1 files changed, 17 insertions, 0 deletions

diff --git a/test/functional/api/window_spec.lua b/test/functional/api/window_spec.lua
index d90323181c..97abcc1e9e 100644
--- a/test/functional/api/window_spec.lua
+++ b/test/functional/api/window_spec.lua
@@ -7,6 +7,8 @@ local clear, nvim, curbuf, curbuf_contents, window, curwin, eq, neq,
 local wait = helpers.wait
 local curwinmeths = helpers.curwinmeths
 local funcs = helpers.funcs
+local request = helpers.request
+local NIL = helpers.NIL
 
 -- check if str is visible at the beginning of some line
 local function is_visible(str)
@@ -137,6 +139,21 @@ describe('window_* functions', function()
       curwinmeths.del_var('lua')
       eq(0, funcs.exists('w:lua'))
     end)
+
+    it('window_set_var returns the old value', function()
+      local val1 = {1, 2, {['3'] = 1}}
+      local val2 = {4, 7}
+      eq(NIL, request('window_set_var', 0, 'lua', val1))
+      eq(val1, request('window_set_var', 0, 'lua', val2))
+    end)
+
+    it('window_del_var returns the old value', function()
+      local val1 = {1, 2, {['3'] = 1}}
+      local val2 = {4, 7}
+      eq(NIL,  request('window_set_var', 0, 'lua', val1))
+      eq(val1, request('window_set_var', 0, 'lua', val2))
+      eq(val2, request('window_del_var', 0, 'lua'))
+    end)
   end)
 
   describe('{get,set}_option', function()