make path mandatory

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

Author: BoazBD

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

@@ -207,7 +207,7 @@ async def get(
 async def mget(
     client: TGlideClient,
     keys: List[TEncodable],
-    path: Optional[TEncodable] = None,
+    path: TEncodable,
 ) -> List[Optional[bytes]]:
     """
     Retrieves the JSON values at the specified `path` stored at multiple `keys`.
@@ -223,7 +223,7 @@ async def mget(
     Args:
         client (TGlideClient): The client to execute the command.
         keys (List[TEncodable]): A list of keys for the JSON documents.
-        path (Optional[TEncodable]): The path within the JSON documents. Default is root `$`.
+        path (TEncodable): The path within the JSON documents.
 
     Returns:
         List[Optional[bytes]]:
@@ -247,10 +247,7 @@ async def mget(
         >>> await glideJson.mget(client, ["doc1"], "$.non_existing_path")
             [None]  # Returns an empty array since the path '$.non_existing_path' does not exist in the JSON document stored at `doc1`.
     """
-    args = ["JSON.MGET"] + keys
-    if path:
-        args.append(path)
-
+    args = ["JSON.MGET"] + keys + [path]
     return cast(TJsonResponse[Optional[bytes]], await client.custom_command(args))
 
 

python/python/tests/tests_server_modules/test_json.py

@@ -305,13 +305,6 @@ async def test_json_mget(self, glide_client: TGlideClient):
         expected_result = [b"[1.0]"]
         assert result == expected_result
 
-        # No path given
-        result = await json.mget(
-            glide_client,
-            [key1, key2],
-        )
-        assert result == [None]
-
     @pytest.mark.parametrize("cluster_mode", [True, False])
     @pytest.mark.parametrize("protocol", [ProtocolVersion.RESP2, ProtocolVersion.RESP3])
     async def test_json_del(self, glide_client: TGlideClient):