bugfix: views: Left strip search text to avoid empty search results.

Tests amended.

Author: preetmishra

tests/ui/test_ui_tools.py

@@ -418,6 +418,8 @@ def test_init(self, mocker, stream_view):
         ('test', ['test here'], []),
         ('here', ['test here'], []),
         ('test here', ['test here'], []),
+        (' ', ['bar', 'boo', 'BOO', 'fan', 'FOO', 'foo', 'FOOBAR',
+               'test here'], []),
         # With 'foo' pinned.
         ('f', ['foo', 'fan', 'FOO', 'FOOBAR'], [['foo'], ]),
         ('FOO', ['foo', 'FOO', 'FOOBAR'], [['foo'], ]),
@@ -546,6 +548,8 @@ def test_init(self, mocker, topic_view):
         ('FOO', ['FOO', 'FOOBAR', 'foo']),
         ('(no', ['(no topic)']),
         ('topic', ['(no topic)']),
+        (' ', ['FOO', 'FOOBAR', 'foo', 'fan', 'boo', 'BOO', 'bar',
+               '(no topic)']),
     ])
     def test_update_topics(self, mocker, topic_view, new_text, expected_log):
         topic_names = [
@@ -979,9 +983,10 @@ def test_update_user_list_editor_mode(self, mocker, right_col_view):
     @pytest.mark.parametrize(['search_string', 'assert_list',
                               'match_return_value'], [
         ('U', ["USER1", "USER2"], True),
-        ('F', [], False)
+        ('F', [], False),
+        (' ', ["USER1", "USER2"], True),
     ], ids=[
-        'user match', 'no user match',
+        'user match', 'no user match', 'no search',
     ])
     def test_update_user_list(self, right_col_view, mocker,
                               search_string, assert_list, match_return_value):
@@ -1002,7 +1007,6 @@ def test_update_user_presence(self, right_col_view, mocker,
         set_body = mocker.patch(VIEWS + ".urwid.Frame.set_body")
 
         right_col_view.update_user_list(user_list=user_list)
-
         right_col_view.users_view.assert_called_with(user_list)
         set_body.assert_called_once_with(right_col_view.body)
 

zulipterminal/ui_tools/views.py

@@ -264,6 +264,7 @@ def __init__(self, streams_btn_list: List[Any], view: Any) -> None:
 
     @asynch
     def update_streams(self, search_box: Any, new_text: str) -> None:
+        new_text = new_text.lstrip()
         if not self.view.controller.is_in_editor_mode():
             return
         # wait for any previously started search to finish to avoid
@@ -331,6 +332,7 @@ def __init__(self, topics_btn_list: List[Any], view: Any,
 
     @asynch
     def update_topics(self, search_box: Any, new_text: str) -> None:
+        new_text = new_text.lstrip()
         if not self.view.controller.is_in_editor_mode():
             return
         # wait for any previously started search to finish to avoid
@@ -563,6 +565,7 @@ def __init__(self, width: int, view: Any) -> None:
     def update_user_list(self, search_box: Any=None,
                          new_text: str="",
                          user_list: Any=None) -> None:
+        new_text = new_text.lstrip()
 
         assert ((user_list is None and search_box is not None)
                 or (user_list is not None and search_box is None