penguinolog / urwid / 42068305391

93%

master: 94%

push

github

Empty a FilteredListWalker when the predicate raises on a source change

After a change to the source the previous visible positions may no longer exist there,
so keeping them let get_focus() and a ListBox read the wrong widget or raise IndexError.

- The view empties itself, emits "modified", and re-raises to the code that changed the source.
- It keeps the widget focused last, so the next change to the source or the predicate returns to it.
- Only the predicate is guarded: a listener of the view that raises leaves the rebuilt view in place.

6652 of 7338 branches covered (90.65%)

Branch coverage included in aggregate %.

14 of 14 new or added lines in 1 file covered. (100.0%)

17080 of 18147 relevant lines covered (94.12%)

14.91 hits per line