tests: shell-test: use count for REP (#10514)

Also fix V576: use width specification

> Incorrect format. Consider checking the third actual argument of the
> 'sscanf' function. It's dangerous to use string specifier without width
> specification. Buffer overflow is possible.

1 files changed, 3 insertions, 4 deletions

diff --git a/test/functional/terminal/edit_spec.lua b/test/functional/terminal/edit_spec.lua
index 84d7ae6e9c..d213bae7b3 100644
--- a/test/functional/terminal/edit_spec.lua
+++ b/test/functional/terminal/edit_spec.lua
@@ -34,10 +34,9 @@ describe(':edit term://*', function()
   it("runs TermOpen early enough to set buffer-local 'scrollback'", function()
     local columns, lines = 20, 4
     local scr = get_screen(columns, lines)
-    local rep = 'a'
+    local rep = 97
     meths.set_option('shellcmdflag', 'REP ' .. rep)
     command('set shellxquote=')  -- win: avoid extra quotes
-    local rep_size = rep:byte()  -- 'a' => 97
     local sb = 10
     command('autocmd TermOpen * :setlocal scrollback='..tostring(sb)
             ..'|call feedkeys("G", "n")')
@@ -45,8 +44,8 @@ describe(':edit term://*', function()
 
     local bufcontents = {}
     local winheight = curwinmeths.get_height()
-    local buf_cont_start = rep_size - sb - winheight + 2
-    for i = buf_cont_start,(rep_size - 1) do
+    local buf_cont_start = rep - sb - winheight + 2
+    for i = buf_cont_start,(rep - 1) do
       bufcontents[#bufcontents + 1] = ('%d: foobar'):format(i)
     end
     bufcontents[#bufcontents + 1] = ''