Handle lists in the response of INFO (#3278)

Parse lists in the response of INFO, and even lines where list items are
mixed with key=value items, in which case the overall structure will be
a dict, and the items without value get `True` as their value.

Author: gerzse

redis/_parsers/helpers.py

@@ -46,11 +46,18 @@ def get_value(value):
                     return int(value)
             except ValueError:
                 return value
+        elif "=" not in value:
+            return [get_value(v) for v in value.split(",") if v]
         else:
             sub_dict = {}
             for item in value.split(","):
-                k, v = item.rsplit("=", 1)
-                sub_dict[k] = get_value(v)
+                if not item:
+                    continue
+                if "=" in item:
+                    k, v = item.rsplit("=", 1)
+                    sub_dict[k] = get_value(v)
+                else:
+                    sub_dict[item] = True
             return sub_dict
 
     for line in response.splitlines():

tests/test_parsers/test_helpers.py

@@ -33,3 +33,31 @@ def test_parse_info():
     assert info["search_version"] == "99.99.99"
     assert info["search_redis_version"] == "7.2.2 - oss"
     assert info["search_query_timeout_ms"] == 500
+
+
+def test_parse_info_list():
+    info_output = """
+list_one:a,
+list_two:a b,,c,10,1.1
+    """
+    info = parse_info(info_output)
+
+    assert isinstance(info["list_one"], list)
+    assert info["list_one"] == ["a"]
+
+    assert isinstance(info["list_two"], list)
+    assert info["list_two"] == ["a b", "c", 10, 1.1]
+
+
+def test_parse_info_list_dict_mixed():
+    info_output = """
+list_one:a,b=1
+list_two:a b=foo,,c,d=bar,e,
+    """
+    info = parse_info(info_output)
+
+    assert isinstance(info["list_one"], dict)
+    assert info["list_one"] == {"a": True, "b": 1}
+
+    assert isinstance(info["list_two"], dict)
+    assert info["list_two"] == {"a b": "foo", "c": True, "d": "bar", "e": True}