From d835c030f24a598ef02815dfd21dd40307f9a22c Mon Sep 17 00:00:00 2001
From: Paul Jolly <paul@myitcv.org.uk>
Date: Fri, 28 Aug 2015 18:55:17 +0100
Subject: remote/define.vim: support remote function "range"

---
 runtime/autoload/remote/define.vim | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/runtime/autoload/remote/define.vim b/runtime/autoload/remote/define.vim
index ca1def0941..dcf773d215 100644
--- a/runtime/autoload/remote/define.vim
+++ b/runtime/autoload/remote/define.vim
@@ -1,7 +1,7 @@
 function! remote#define#CommandOnHost(host, method, sync, name, opts)
   let prefix = ''
 
-  if has_key(a:opts, 'range') 
+  if has_key(a:opts, 'range')
     if a:opts.range == '' || a:opts.range == '%'
       " -range or -range=%, pass the line range in a list
       let prefix = '<line1>,<line2>'
@@ -157,6 +157,9 @@ endfunction
 
 function! remote#define#FunctionOnChannel(channel, method, sync, name, opts)
   let rpcargs = [a:channel, '"'.a:method.'"', 'a:000']
+  if has_key(a:opts, 'range') && a:opts.range != '0'
+    call add(rpcargs, '[a:firstline, a:lastline]')
+  endif
   call s:AddEval(rpcargs, a:opts)
 
   let function_def = s:GetFunctionPrefix(a:name, a:opts)
@@ -187,7 +190,7 @@ let s:next_gid = 1
 function! s:GetNextAutocmdGroup()
   let gid = s:next_gid
   let s:next_gid += 1
-  
+
   let group_name = 'RPC_DEFINE_AUTOCMD_GROUP_'.gid
   " Ensure the group is defined
   exe 'augroup '.group_name.' | augroup END'
@@ -218,7 +221,11 @@ endfunction
 
 
 function! s:GetFunctionPrefix(name, opts)
-  return "function! ".a:name."(...)\n"
+  let res = "function! ".a:name."(...)"
+  if has_key(a:opts, 'range') && a:opts.range != '0'
+    let res = res." range"
+  endif
+  return res."\n"
 endfunction
 
 
-- 
cgit 


From 4a200ceafa475f5c4a7dc576df97a12caedd3590 Mon Sep 17 00:00:00 2001
From: Case Nelson <case@outpace.com>
Date: Sun, 10 Apr 2016 00:20:57 -0600
Subject: test: remote/define.vim: range

---
 runtime/autoload/remote/define.vim       |  4 ++--
 test/functional/provider/define_spec.lua | 33 ++++++++++++++++++++++++++++++++
 2 files changed, 35 insertions(+), 2 deletions(-)

diff --git a/runtime/autoload/remote/define.vim b/runtime/autoload/remote/define.vim
index dcf773d215..b04a5d2280 100644
--- a/runtime/autoload/remote/define.vim
+++ b/runtime/autoload/remote/define.vim
@@ -157,7 +157,7 @@ endfunction
 
 function! remote#define#FunctionOnChannel(channel, method, sync, name, opts)
   let rpcargs = [a:channel, '"'.a:method.'"', 'a:000']
-  if has_key(a:opts, 'range') && a:opts.range != '0'
+  if has_key(a:opts, 'range')
     call add(rpcargs, '[a:firstline, a:lastline]')
   endif
   call s:AddEval(rpcargs, a:opts)
@@ -222,7 +222,7 @@ endfunction
 
 function! s:GetFunctionPrefix(name, opts)
   let res = "function! ".a:name."(...)"
-  if has_key(a:opts, 'range') && a:opts.range != '0'
+  if has_key(a:opts, 'range')
     let res = res." range"
   endif
   return res."\n"
diff --git a/test/functional/provider/define_spec.lua b/test/functional/provider/define_spec.lua
index 6e8a3b89cd..4a8c68350f 100644
--- a/test/functional/provider/define_spec.lua
+++ b/test/functional/provider/define_spec.lua
@@ -333,6 +333,39 @@ local function function_specs_for(fn, sync, first_arg_factory, init)
           runx(sync, handler, on_setup)
         end)
       end)
+
+      describe('with range', function()
+        it('ok', function()
+          call(fn, args..[[, {'range': ''}]])
+          local function on_setup()
+            command('%call TestFunction(1, "a", ["b", "c"])')
+          end
+
+          local function handler(method, arguments)
+            eq('test-handler', method)
+            eq({{1, 'a', {'b', 'c'}}, {1, 1}}, arguments)
+            return 'rv'
+          end
+
+          runx(sync, handler, on_setup)
+        end)
+      end)
+      describe('with eval/range', function()
+        it('ok', function()
+          call(fn, args..[[, {'eval': '4', 'range': ''}]])
+          local function on_setup()
+            command('%call TestFunction(1, "a", ["b", "c"])')
+          end
+
+          local function handler(method, arguments)
+            eq('test-handler', method)
+            eq({{1, 'a', {'b', 'c'}}, {1, 1}, 4}, arguments)
+            return 'rv'
+          end
+
+          runx(sync, handler, on_setup)
+        end)
+      end)
     end)
   end)
 end
-- 
cgit 


From c4117d4b2f0dfa08ee44ac6bd60d7f01e73d824e Mon Sep 17 00:00:00 2001
From: Case Nelson <case@outpace.com>
Date: Sun, 10 Apr 2016 19:47:21 -0600
Subject: test: format

---
 test/functional/provider/define_spec.lua | 373 ++++++++++++++-----------------
 1 file changed, 173 insertions(+), 200 deletions(-)

diff --git a/test/functional/provider/define_spec.lua b/test/functional/provider/define_spec.lua
index 4a8c68350f..26cd997fc3 100644
--- a/test/functional/provider/define_spec.lua
+++ b/test/functional/provider/define_spec.lua
@@ -64,153 +64,137 @@ local function command_specs_for(fn, sync, first_arg_factory, init)
         args = args..', "RpcCommand"'
       end)
 
-      describe('without options', function()
-        it('ok', function()
-          call(fn, args..', {}')
-          local function on_setup()
-            command('RpcCommand')
-          end
+      it('without options', function()
+        call(fn, args..', {}')
+        local function on_setup()
+          command('RpcCommand')
+        end
 
-          local function handler(method)
-            eq('test-handler', method)
-            return ''
-          end
+        local function handler(method)
+          eq('test-handler', method)
+          return ''
+        end
 
-          runx(sync, handler, on_setup)
-        end)
+        runx(sync, handler, on_setup)
       end)
 
-      describe('with nargs', function()
-        it('ok', function()
-          call(fn, args..', {"nargs": "*"}')
-          local function on_setup()
-            command('RpcCommand arg1 arg2 arg3')
-          end
+      it('with nargs', function()
+        call(fn, args..', {"nargs": "*"}')
+        local function on_setup()
+          command('RpcCommand arg1 arg2 arg3')
+        end
 
-          local function handler(method, arguments)
-            eq('test-handler', method)
-            eq({'arg1', 'arg2', 'arg3'}, arguments[1])
-            return ''
-          end
+        local function handler(method, arguments)
+          eq('test-handler', method)
+          eq({'arg1', 'arg2', 'arg3'}, arguments[1])
+          return ''
+        end
 
-          runx(sync, handler, on_setup)
-        end)
+        runx(sync, handler, on_setup)
       end)
 
-      describe('with range', function()
-        it('ok', function()
-          call(fn,args..', {"range": ""}')
-          local function on_setup()
-            command('1,1RpcCommand')
-          end
+      it('with range', function()
+        call(fn,args..', {"range": ""}')
+        local function on_setup()
+          command('1,1RpcCommand')
+        end
 
-          local function handler(method, arguments)
-            eq('test-handler', method)
-            eq({1, 1}, arguments[1])
-            return ''
-          end
+        local function handler(method, arguments)
+          eq('test-handler', method)
+          eq({1, 1}, arguments[1])
+          return ''
+        end
 
-          runx(sync, handler, on_setup)
-        end)
+        runx(sync, handler, on_setup)
       end)
 
-      describe('with nargs/range', function()
-        it('ok', function()
-          call(fn, args..', {"nargs": "1", "range": ""}')
-          local function on_setup()
-            command('1,1RpcCommand arg')
-          end
+      it('with nargs/range', function()
+        call(fn, args..', {"nargs": "1", "range": ""}')
+        local function on_setup()
+          command('1,1RpcCommand arg')
+        end
 
-          local function handler(method, arguments)
-            eq('test-handler', method)
-            eq({'arg'}, arguments[1])
-            eq({1, 1}, arguments[2])
-            return ''
-          end
+        local function handler(method, arguments)
+          eq('test-handler', method)
+          eq({'arg'}, arguments[1])
+          eq({1, 1}, arguments[2])
+          return ''
+        end
 
-          runx(sync, handler, on_setup)
-        end)
+        runx(sync, handler, on_setup)
       end)
 
-      describe('with nargs/count', function()
-        it('ok', function()
-          call(fn, args..', {"nargs": "1", "range": "5"}')
-          local function on_setup()
-            command('5RpcCommand arg')
-          end
+      it('with nargs/count', function()
+        call(fn, args..', {"nargs": "1", "range": "5"}')
+        local function on_setup()
+          command('5RpcCommand arg')
+        end
 
-          local function handler(method, arguments)
-            eq('test-handler', method)
-            eq({'arg'}, arguments[1])
-            eq(5, arguments[2])
-            return ''
-          end
+        local function handler(method, arguments)
+          eq('test-handler', method)
+          eq({'arg'}, arguments[1])
+          eq(5, arguments[2])
+          return ''
+        end
 
-          runx(sync, handler, on_setup)
-        end)
+        runx(sync, handler, on_setup)
       end)
 
-      describe('with nargs/count/bang', function()
-        it('ok', function()
-          call(fn, args..', {"nargs": "1", "range": "5", "bang": ""}')
-          local function on_setup()
-            command('5RpcCommand! arg')
-          end
+      it('with nargs/count/bang', function()
+        call(fn, args..', {"nargs": "1", "range": "5", "bang": ""}')
+        local function on_setup()
+          command('5RpcCommand! arg')
+        end
 
-          local function handler(method, arguments)
-            eq('test-handler', method)
-            eq({'arg'}, arguments[1])
-            eq(5, arguments[2])
-            eq(1, arguments[3])
-            return ''
-          end
+        local function handler(method, arguments)
+          eq('test-handler', method)
+          eq({'arg'}, arguments[1])
+          eq(5, arguments[2])
+          eq(1, arguments[3])
+          return ''
+        end
 
-          runx(sync, handler, on_setup)
-        end)
+        runx(sync, handler, on_setup)
       end)
 
-      describe('with nargs/count/bang/register', function()
-        it('ok', function()
-          call(fn, args..', {"nargs": "1", "range": "5", "bang": "",'..
-                     ' "register": ""}')
-          local function on_setup()
-            command('5RpcCommand! b arg')
-          end
+      it('with nargs/count/bang/register', function()
+        call(fn, args..', {"nargs": "1", "range": "5", "bang": "",'..
+        ' "register": ""}')
+        local function on_setup()
+          command('5RpcCommand! b arg')
+        end
 
-          local function handler(method, arguments)
-            eq('test-handler', method)
-            eq({'arg'}, arguments[1])
-            eq(5, arguments[2])
-            eq(1, arguments[3])
-            eq('b', arguments[4])
-            return ''
-          end
+        local function handler(method, arguments)
+          eq('test-handler', method)
+          eq({'arg'}, arguments[1])
+          eq(5, arguments[2])
+          eq(1, arguments[3])
+          eq('b', arguments[4])
+          return ''
+        end
 
-          runx(sync, handler, on_setup)
-        end)
+        runx(sync, handler, on_setup)
       end)
 
-      describe('with nargs/count/bang/register/eval', function()
-        it('ok', function()
-          call(fn, args..', {"nargs": "1", "range": "5", "bang": "",'..
-                     ' "register": "", "eval": "@<reg>"}')
-          local function on_setup()
-            command('let @b = "regb"')
-            command('5RpcCommand! b arg')
-          end
+      it('with nargs/count/bang/register/eval', function()
+        call(fn, args..', {"nargs": "1", "range": "5", "bang": "",'..
+        ' "register": "", "eval": "@<reg>"}')
+        local function on_setup()
+          command('let @b = "regb"')
+          command('5RpcCommand! b arg')
+        end
 
-          local function handler(method, arguments)
-            eq('test-handler', method)
-            eq({'arg'}, arguments[1])
-            eq(5, arguments[2])
-            eq(1, arguments[3])
-            eq('b', arguments[4])
-            eq('regb', arguments[5])
-            return ''
-          end
+        local function handler(method, arguments)
+          eq('test-handler', method)
+          eq({'arg'}, arguments[1])
+          eq(5, arguments[2])
+          eq(1, arguments[3])
+          eq('b', arguments[4])
+          eq('regb', arguments[5])
+          return ''
+        end
 
-          runx(sync, handler, on_setup)
-        end)
+        runx(sync, handler, on_setup)
       end)
     end)
   end)
@@ -236,37 +220,33 @@ local function autocmd_specs_for(fn, sync, first_arg_factory, init)
         args = args..', "BufEnter"'
       end)
 
-      describe('without options', function()
-        it('ok', function()
-          call(fn, args..', {}')
-          local function on_setup()
-            command('doautocmd BufEnter x.c')
-          end
+      it('without options', function()
+        call(fn, args..', {}')
+        local function on_setup()
+          command('doautocmd BufEnter x.c')
+        end
 
-          local function handler(method)
-            eq('test-handler', method)
-            return ''
-          end
+        local function handler(method)
+          eq('test-handler', method)
+          return ''
+        end
 
-          runx(sync, handler, on_setup)
-        end)
+        runx(sync, handler, on_setup)
       end)
 
-      describe('with eval', function()
-        it('ok', function()
-          call(fn, args..[[, {'eval': 'expand("<afile>")'}]])
-          local function on_setup()
-            command('doautocmd BufEnter x.c')
-          end
+      it('with eval', function()
+        call(fn, args..[[, {'eval': 'expand("<afile>")'}]])
+        local function on_setup()
+          command('doautocmd BufEnter x.c')
+        end
 
-          local function handler(method, arguments)
-            eq('test-handler', method)
-            eq('x.c', arguments[1])
-            return ''
-          end
+        local function handler(method, arguments)
+          eq('test-handler', method)
+          eq('x.c', arguments[1])
+          return ''
+        end
 
-          runx(sync, handler, on_setup)
-        end)
+        runx(sync, handler, on_setup)
       end)
     end)
   end)
@@ -292,79 +272,72 @@ local function function_specs_for(fn, sync, first_arg_factory, init)
         args = args..', "TestFunction"'
       end)
 
-      describe('without options', function()
-        it('ok', function()
-          call(fn, args..', {}')
-          local function on_setup()
-            if sync then
-              eq('rv', eval('TestFunction(1, "a", ["b", "c"])'))
-            else
-              eq(1, eval('TestFunction(1, "a", ["b", "c"])'))
-            end
+      it('without options', function()
+        call(fn, args..', {}')
+        local function on_setup()
+          if sync then
+            eq('rv', eval('TestFunction(1, "a", ["b", "c"])'))
+          else
+            eq(1, eval('TestFunction(1, "a", ["b", "c"])'))
           end
+        end
 
-          local function handler(method, arguments)
-            eq('test-handler', method)
-            eq({{1, 'a', {'b', 'c'}}}, arguments)
-            return 'rv'
-          end
+        local function handler(method, arguments)
+          eq('test-handler', method)
+          eq({{1, 'a', {'b', 'c'}}}, arguments)
+          return 'rv'
+        end
 
-          runx(sync, handler, on_setup)
-        end)
+        runx(sync, handler, on_setup)
       end)
 
-      describe('with eval', function()
-        it('ok', function()
-          call(fn, args..[[, {'eval': '2 + 2'}]])
-          local function on_setup()
-            if sync then
-              eq('rv', eval('TestFunction(1, "a", ["b", "c"])'))
-            else
-              eq(1, eval('TestFunction(1, "a", ["b", "c"])'))
-            end
+      it('with eval', function()
+        call(fn, args..[[, {'eval': '2 + 2'}]])
+        local function on_setup()
+          if sync then
+            eq('rv', eval('TestFunction(1, "a", ["b", "c"])'))
+          else
+            eq(1, eval('TestFunction(1, "a", ["b", "c"])'))
           end
+        end
 
-          local function handler(method, arguments)
-            eq('test-handler', method)
-            eq({{1, 'a', {'b', 'c'}}, 4}, arguments)
-            return 'rv'
-          end
+        local function handler(method, arguments)
+          eq('test-handler', method)
+          eq({{1, 'a', {'b', 'c'}}, 4}, arguments)
+          return 'rv'
+        end
 
-          runx(sync, handler, on_setup)
-        end)
+        runx(sync, handler, on_setup)
       end)
 
-      describe('with range', function()
-        it('ok', function()
-          call(fn, args..[[, {'range': ''}]])
-          local function on_setup()
-            command('%call TestFunction(1, "a", ["b", "c"])')
-          end
+      it('with range', function()
+        call(fn, args..[[, {'range': ''}]])
+        local function on_setup()
+          command('%call TestFunction(1, "a", ["b", "c"])')
+        end
 
-          local function handler(method, arguments)
-            eq('test-handler', method)
-            eq({{1, 'a', {'b', 'c'}}, {1, 1}}, arguments)
-            return 'rv'
-          end
+        local function handler(method, arguments)
+          eq('test-handler', method)
+          eq({{1, 'a', {'b', 'c'}}, {1, 1}}, arguments)
+          return 'rv'
+        end
 
-          runx(sync, handler, on_setup)
-        end)
+        runx(sync, handler, on_setup)
       end)
-      describe('with eval/range', function()
-        it('ok', function()
-          call(fn, args..[[, {'eval': '4', 'range': ''}]])
-          local function on_setup()
-            command('%call TestFunction(1, "a", ["b", "c"])')
-          end
 
-          local function handler(method, arguments)
-            eq('test-handler', method)
-            eq({{1, 'a', {'b', 'c'}}, {1, 1}, 4}, arguments)
-            return 'rv'
-          end
+      it('with eval/range', function()
+        call(fn, args..[[, {'eval': '4', 'range': ''}]])
+        local function on_setup()
+          command('%call TestFunction(1, "a", ["b", "c"])')
+        end
+
+        local function handler(method, arguments)
+          eq('test-handler', method)
+          eq({{1, 'a', {'b', 'c'}}, {1, 1}, 4}, arguments)
+          return 'rv'
+        end
 
-          runx(sync, handler, on_setup)
-        end)
+        runx(sync, handler, on_setup)
       end)
     end)
   end)
-- 
cgit 


From a347b29aa3caf245a99e12bfc8167f3bf882f770 Mon Sep 17 00:00:00 2001
From: "Justin M. Keyes" <justinkz@gmail.com>
Date: Mon, 18 Apr 2016 05:39:18 -0400
Subject: test: remote/define.vim: Add some spice.

---
 test/functional/provider/define_spec.lua | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/test/functional/provider/define_spec.lua b/test/functional/provider/define_spec.lua
index 26cd997fc3..c30ad6d8c2 100644
--- a/test/functional/provider/define_spec.lua
+++ b/test/functional/provider/define_spec.lua
@@ -311,14 +311,19 @@ local function function_specs_for(fn, sync, first_arg_factory, init)
       end)
 
       it('with range', function()
+        helpers.insert([[
+          foo
+          bar
+          baz
+          zub]])
         call(fn, args..[[, {'range': ''}]])
         local function on_setup()
-          command('%call TestFunction(1, "a", ["b", "c"])')
+          command('2,3call TestFunction(1, "a", ["b", "c"])')
         end
 
         local function handler(method, arguments)
           eq('test-handler', method)
-          eq({{1, 'a', {'b', 'c'}}, {1, 1}}, arguments)
+          eq({{1, 'a', {'b', 'c'}}, {2, 3}}, arguments)
           return 'rv'
         end
 
-- 
cgit