From e5c4c4bca20678e6c621847f370701945ce625db Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bj=C3=B6rn=20Linse?= Date: Sun, 22 Mar 2015 14:18:35 +0100 Subject: [PATCH 1/3] test/ui: add redraw debug util --- test/functional/ui/screen.lua | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/test/functional/ui/screen.lua b/test/functional/ui/screen.lua index 174538df6e..6c23611c07 100644 --- a/test/functional/ui/screen.lua +++ b/test/functional/ui/screen.lua @@ -456,6 +456,24 @@ end function Screen:snapshot_util(attrs, ignore) -- util to generate screen test pcall(function() self:wait(function() return "error" end, 250) end) + self:print_snapshot(attrs, ignore) +end + +function Screen:redraw_debug(attrs, ignore) + self:print_snapshot(attrs, ignore) + local function notification_cb(method, args) + assert(method == 'redraw') + for _, update in ipairs(args) do + print(require('inspect')(update)) + end + self:_redraw(args) + self:print_snapshot(attrs, ignore) + return true + end + run(nil, notification_cb, nil, 250) +end + +function Screen:print_snapshot(attrs, ignore) if ignore == nil then ignore = self._default_attr_ignore end From 1e7624d2a0a4be9a7e898994728064eb88a91576 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bj=C3=B6rn=20Linse?= Date: Sun, 22 Mar 2015 15:23:53 +0100 Subject: [PATCH 2/3] tests/ui: warn when the correct state was seen temporarily. Helped-by: Thiago de Arruda --- test/functional/ui/screen.lua | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/test/functional/ui/screen.lua b/test/functional/ui/screen.lua index 6c23611c07..ca6cac1ba3 100644 --- a/test/functional/ui/screen.lua +++ b/test/functional/ui/screen.lua @@ -230,20 +230,41 @@ end function Screen:wait(check, timeout) local err, checked = false + local success_seen = false + local failure_after_success = false local function notification_cb(method, args) assert(method == 'redraw') self:_redraw(args) err = check() checked = true if not err then + success_seen = true stop() + elseif success_seen and #args > 0 then + failure_after_success = true + --print(require('inspect')(args)) end + return true end run(nil, notification_cb, nil, timeout or default_screen_timeout) if not checked then err = check() end + + if failure_after_success then + print([[ +Warning: Screen changes have been received after the expected state was seen. +This is probably due to an indeterminism in the test. Try adding +`wait()` (or even a separate `screen:expect(...)`) at a point of possible +indeterminism, typically in between a `feed()` or `execute()` which is non- +synchronous, and a synchronous api call. + ]]) + local tb = debug.traceback() + local index = string.find(tb, '\n%s*%[C]') + print(string.sub(tb,1,index)) + end + if err then assert(false, err) end From 3a20a9847eb89e5bec84e7564ac88ab35b6c7b1c Mon Sep 17 00:00:00 2001 From: Thiago de Arruda Date: Sun, 22 Mar 2015 15:55:10 +0100 Subject: [PATCH 3/3] test/ui: Fix indeterminism in screen_basic_spec --- test/functional/ui/screen_basic_spec.lua | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/test/functional/ui/screen_basic_spec.lua b/test/functional/ui/screen_basic_spec.lua index 034e9a05d7..7710918b94 100644 --- a/test/functional/ui/screen_basic_spec.lua +++ b/test/functional/ui/screen_basic_spec.lua @@ -1,7 +1,7 @@ local helpers = require('test.functional.helpers') local Screen = require('test.functional.ui.screen') local clear, feed, execute = helpers.clear, helpers.feed, helpers.execute -local insert = helpers.insert +local insert, wait = helpers.insert, helpers.wait describe('Screen', function() local screen @@ -464,6 +464,7 @@ describe('Screen', function() end) it('has minimum width/height values', function() + wait() screen:try_resize(1, 1) screen:expect([[ -- INS^ERT --|