Improve documentation and tests

Signed-off-by: BoazBD <[email protected]>

Author: BoazBD

python/python/glide/async_commands/server_modules/json.py

@@ -227,7 +227,6 @@ async def mget(
 
     Returns:
         List[Optional[bytes]]:
-            List[Optional[bytes]]:
             For JSONPath (`path` starts with `$`):
                 Returns a list of byte representations of the values found at the given path for each key.
                 If `path` does not exist within the key, the entry will be an empty array.

python/python/tests/tests_server_modules/test_json.py

@@ -237,6 +237,15 @@ async def test_json_mget(self, glide_client: TGlideClient):
         expected_result = [b"2.5", b"4"]
         assert result == expected_result
 
+        # JSONPath that exists in only one of the keys
+        result = await json.mget(
+            glide_client,
+            [key1, key2],
+            "$.b.c",
+        )
+        expected_result = [b"[true]", b"[]"]
+        assert result == expected_result
+
         # Legacy path that exists in only one of the keys
         result = await json.mget(
             glide_client,
@@ -263,14 +272,22 @@ async def test_json_mget(self, glide_client: TGlideClient):
         )
         assert result == [None, None]
 
-        # One key doesn't exist
+        # JSONPath one key doesn't exist
         result = await json.mget(
             glide_client,
             [key1, "{non_existing_key}"],
             "$.a",
         )
         assert result == [b"[1.0]", None]
 
+        # Legacy path one key doesn't exist
+        result = await json.mget(
+            glide_client,
+            [key1, "{non_existing_key}"],
+            ".a",
+        )
+        assert result == [b"1.0", None]
+
         # Both keys don't exist
         result = await json.mget(
             glide_client,