functests: Replace execute with either command or feed_command

Hope this will make people using feed_command less likely: this hides bugs.
Already found at least two:

1. msgpackparse() will show internal error: hash_add() in case of duplicate
   keys, though it will still work correctly. Currently silenced.
2. ttimeoutlen was spelled incorrectly, resulting in option not being set when
   expected. Test was still functioning somehow though. Currently fixed.

1 files changed, 7 insertions, 7 deletions

diff --git a/test/functional/legacy/signs_spec.lua b/test/functional/legacy/signs_spec.lua
index e80a32455a..0e65edbcf8 100644
--- a/test/functional/legacy/signs_spec.lua
+++ b/test/functional/legacy/signs_spec.lua
@@ -1,23 +1,23 @@
 -- Tests for signs
 
 local helpers = require('test.functional.helpers')(after_each)
-local clear, execute, expect = helpers.clear, helpers.execute, helpers.expect
+local clear, command, expect = helpers.clear, helpers.command, helpers.expect
 
 describe('signs', function()
   setup(clear)
 
   it('is working', function()
-    execute('sign define JumpSign text=x')
-    execute([[exe 'sign place 42 line=2 name=JumpSign buffer=' . bufnr('')]])
+    command('sign define JumpSign text=x')
+    command([[exe 'sign place 42 line=2 name=JumpSign buffer=' . bufnr('')]])
     -- Split the window to the bottom to verify :sign-jump will stay in the current
     -- window if the buffer is displayed there.
-    execute('bot split')
-    execute([[exe 'sign jump 42 buffer=' . bufnr('')]])
-    execute([[call append(line('$'), winnr())]])
+    command('bot split')
+    command([[exe 'sign jump 42 buffer=' . bufnr('')]])
+    command([[call append(line('$'), winnr())]])
 
     -- Assert buffer contents.
     expect([[
-      
+
       2]])
   end)
 end)