From 32f3937477e05b13e05bba4463c5cfd417a2fde8 Mon Sep 17 00:00:00 2001 From: "Justin M. Keyes" Date: Mon, 23 Apr 2018 04:43:00 +0200 Subject: test: fsync() codepaths --- test/functional/core/fileio_spec.lua | 61 ++++++++++++++++++++++++++++++++++++ 1 file changed, 61 insertions(+) create mode 100644 test/functional/core/fileio_spec.lua (limited to 'test') diff --git a/test/functional/core/fileio_spec.lua b/test/functional/core/fileio_spec.lua new file mode 100644 index 0000000000..848ef7cae0 --- /dev/null +++ b/test/functional/core/fileio_spec.lua @@ -0,0 +1,61 @@ +local helpers = require('test.functional.helpers')(after_each) + +local clear = helpers.clear +local command = helpers.command +local eq = helpers.eq +local eval = helpers.eval +local feed = helpers.feed +local funcs = helpers.funcs +local nvim_prog = helpers.nvim_prog +local rmdir = helpers.rmdir +local sleep = helpers.sleep + +describe('fileio', function() + before_each(function() + end) + after_each(function() + command(':qall!') + os.remove('Xtest_startup_shada') + os.remove('Xtest_startup_file1') + os.remove('Xtest_startup_file2') + rmdir('Xtest_startup_swapdir') + end) + + it('fsync() codepaths #8304', function() + -- This is an "acceptance test" or "smoke test". + + clear({ args={ '-i', 'Xtest_startup_shada', + '--cmd', 'set directory=Xtest_startup_swapdir' } }) + + -- These cases ALWAYS force fsync (regardless of 'fsync' option): + + -- 1. Idle (CursorHold) with modified buffers (+ 'swapfile'). + command('set swapfile') + command('set updatetime=1') + command('write Xtest_startup_file1') + feed('ifooh') + sleep(2) + eq(1, eval('&modified')) + + -- 2. Exit caused by deadly signal (+ 'swapfile'). + local j = funcs.jobstart({ nvim_prog, '-u', 'NONE', '-i', + 'Xtest_startup_shada', '--headless', + '-c', 'set swapfile', + '-c', 'write Xtest_startup_file2', + '-c', 'put =localtime()', }) + sleep(10) -- Let Nvim start. + funcs.jobstop(j) -- Send deadly signal. + + -- 3. SIGPWR signal. + -- ?? + + -- 4. Explicit :preserve command. + command('preserve') + + -- 5. Enable 'fsync' option, write file. + command('set fsync') + feed('ibazh') + command('write') + end) +end) + -- cgit From 77cb14cc6da5dff685c6e5a4005da433c39d5ff7 Mon Sep 17 00:00:00 2001 From: "Justin M. Keyes" Date: Tue, 24 Apr 2018 00:27:09 +0200 Subject: API: nvim__stats() Use it to verify fsync() behavior. --- test/functional/core/fileio_spec.lua | 21 ++++++++++++++------- 1 file changed, 14 insertions(+), 7 deletions(-) (limited to 'test') diff --git a/test/functional/core/fileio_spec.lua b/test/functional/core/fileio_spec.lua index 848ef7cae0..09533e4e60 100644 --- a/test/functional/core/fileio_spec.lua +++ b/test/functional/core/fileio_spec.lua @@ -3,10 +3,11 @@ local helpers = require('test.functional.helpers')(after_each) local clear = helpers.clear local command = helpers.command local eq = helpers.eq -local eval = helpers.eval local feed = helpers.feed local funcs = helpers.funcs local nvim_prog = helpers.nvim_prog +local request = helpers.request +local retry = helpers.retry local rmdir = helpers.rmdir local sleep = helpers.sleep @@ -22,20 +23,24 @@ describe('fileio', function() end) it('fsync() codepaths #8304', function() - -- This is an "acceptance test" or "smoke test". - clear({ args={ '-i', 'Xtest_startup_shada', '--cmd', 'set directory=Xtest_startup_swapdir' } }) -- These cases ALWAYS force fsync (regardless of 'fsync' option): -- 1. Idle (CursorHold) with modified buffers (+ 'swapfile'). - command('set swapfile') - command('set updatetime=1') command('write Xtest_startup_file1') feed('ifooh') - sleep(2) - eq(1, eval('&modified')) + command('write') + eq(0, request('nvim__stats').fsync) -- 'nofsync' is the default. + command('set swapfile') + command('set updatetime=1') + feed('izubh') -- File is 'modified'. + sleep(3) -- Allow 'updatetime' to expire. + retry(3, nil, function() + eq(1, request('nvim__stats').fsync) + end) + command('set updatetime=9999') -- 2. Exit caused by deadly signal (+ 'swapfile'). local j = funcs.jobstart({ nvim_prog, '-u', 'NONE', '-i', @@ -51,11 +56,13 @@ describe('fileio', function() -- 4. Explicit :preserve command. command('preserve') + eq(2, request('nvim__stats').fsync) -- 5. Enable 'fsync' option, write file. command('set fsync') feed('ibazh') command('write') + eq(4, request('nvim__stats').fsync) end) end) -- cgit