feat(tck): implement deleteFile JSON-RPC method (hiero-ledger#2571)

Signed-off-by: Siddhartha Ganguly <gangulysiddhartha22@gmail.com>

Author: gangulysiddhartha22-cmyk

src/hiero_sdk_python/file/file_delete_transaction.py

@@ -9,6 +9,7 @@
 from hiero_sdk_python.hapi.services.schedulable_transaction_body_pb2 import (
     SchedulableTransactionBody,
 )
+from hiero_sdk_python.hapi.services.transaction_pb2 import TransactionBody
 from hiero_sdk_python.hbar import Hbar
 from hiero_sdk_python.transaction.transaction import Transaction
 
@@ -19,17 +20,14 @@
 class FileDeleteTransaction(Transaction):
     """
     Represents a file deletion transaction on the network.
-
     This transaction deletes a specified file, rendering it inactive.
-
     Inherits from the base Transaction class and implements the required methods
     to build and execute a file deletion transaction.
     """
 
     def __init__(self, file_id: FileId | None = None):
         """
         Initializes a new FileDeleteTransaction instance with optional file_id.
-
         Args:
             file_id (FileId, optional): The ID of the file to be deleted.
         """
@@ -40,36 +38,27 @@ def __init__(self, file_id: FileId | None = None):
     def set_file_id(self, file_id: FileId) -> FileDeleteTransaction:
         """
         Sets the ID of the file to be deleted.
-
         Args:
             file_id (FileId): The ID of the file to be deleted.
-
         Returns:
             FileDeleteTransaction: Returns self for method chaining.
         """
         self._require_not_frozen()
         self.file_id = file_id
         return self
 
-    def _build_proto_body(self):
+    def _build_proto_body(self) -> FileDeleteTransactionBody:
         """
         Returns the protobuf body for the file delete transaction.
 
         Returns:
             FileDeleteTransactionBody: The protobuf body for this transaction.
-
-        Raises:
-            ValueError: If file_id is not set.
         """
-        if self.file_id is None:
-            raise ValueError("Missing required FileID")
+        return FileDeleteTransactionBody(fileID=self.file_id._to_proto() if self.file_id is not None else None)
 
-        return FileDeleteTransactionBody(fileID=self.file_id._to_proto())
-
-    def build_transaction_body(self):
+    def build_transaction_body(self) -> TransactionBody:
         """
         Builds and returns the protobuf transaction body for file deletion.
-
         Returns:
             TransactionBody: The protobuf transaction body containing the file deletion details.
         """
@@ -81,7 +70,6 @@ def build_transaction_body(self):
     def build_scheduled_body(self) -> SchedulableTransactionBody:
         """
         Builds the scheduled transaction body for this file delete transaction.
-
         Returns:
             SchedulableTransactionBody: The built scheduled transaction body.
         """
@@ -93,13 +81,10 @@ def build_scheduled_body(self) -> SchedulableTransactionBody:
     def _get_method(self, channel: _Channel) -> _Method:
         """
         Gets the method to execute the file delete transaction.
-
         This internal method returns a _Method object containing the appropriate gRPC
         function to call when executing this transaction on the network.
-
         Args:
             channel (_Channel): The channel containing service stubs
-
         Returns:
             _Method: An object containing the transaction function to delete a file.
         """

tck/handlers/file.py

@@ -2,6 +2,7 @@
 
 from hiero_sdk_python.file.file_contents_query import FileContentsQuery
 from hiero_sdk_python.file.file_create_transaction import FileCreateTransaction
+from hiero_sdk_python.file.file_delete_transaction import FileDeleteTransaction
 from hiero_sdk_python.file.file_id import FileId
 from hiero_sdk_python.file.file_info import FileInfo
 from hiero_sdk_python.file.file_info_query import FileInfoQuery
@@ -11,8 +12,8 @@
 from hiero_sdk_python.transaction.transaction_receipt import TransactionReceipt
 from tck.errors import JsonRpcError
 from tck.handlers.registry import rpc_method
-from tck.param.file import CreateFileParams, GetFileContentsParams, GetFileInfoParams
-from tck.response.file import CreateFileResponse, GetFileContentsResponse, GetFileInfoResponse
+from tck.param.file import CreateFileParams, DeleteFileParams, GetFileContentsParams, GetFileInfoParams
+from tck.response.file import CreateFileResponse, DeleteFileResponse, GetFileContentsResponse, GetFileInfoResponse
 from tck.util.client_utils import get_client
 from tck.util.constants import DEFAULT_GRPC_TIMEOUT
 from tck.util.key_utils import get_key_from_string, key_to_string
@@ -106,3 +107,22 @@ def get_file_info(params: GetFileInfoParams) -> GetFileInfoResponse:
 
     info = query.execute(client)
     return _build_file_info_response(info)
+
+
+@rpc_method("deleteFile")
+def delete_file(params: DeleteFileParams) -> DeleteFileResponse:
+    """Delete a file."""
+    client = get_client(params.sessionId)
+
+    transaction = FileDeleteTransaction().set_grpc_deadline(DEFAULT_GRPC_TIMEOUT)
+
+    if params.fileId is not None:
+        transaction.set_file_id(FileId.from_string(params.fileId))
+
+    if params.commonTransactionParams is not None:
+        params.commonTransactionParams.apply_common_params(transaction, client)
+
+    response = transaction.execute(client, wait_for_receipt=False)
+    receipt: TransactionReceipt = response.get_receipt(client, validate_status=True)
+
+    return DeleteFileResponse(ResponseCode(receipt.status).name)

tck/param/file.py

@@ -45,6 +45,7 @@ class GetFileContentsParams(BaseParams):
 
     @classmethod
     def parse_json_params(cls, params: dict) -> GetFileContentsParams:
+        """Parse JSON-RPC params into a GetFileContentsParams instance."""
         return cls(
             sessionId=parse_session_id(params),
             fileId=params.get("fileId"),
@@ -63,3 +64,19 @@ class GetFileInfoParams(BaseParams):
     def parse_json_params(cls, params: dict) -> GetFileInfoParams:
         """Parse JSON-RPC params into a GetFileInfoParams instance."""
         return cls(fileId=params.get("fileId"), sessionId=parse_session_id(params))
+
+
+@dataclass
+class DeleteFileParams(BaseTransactionParams):
+    """Parameters for deleting a file. Extends BaseTransactionParams to include common transaction parameters."""
+
+    fileId: str | None = None
+
+    @classmethod
+    def parse_json_params(cls, params: dict) -> DeleteFileParams:
+
+        return cls(
+            fileId=params.get("fileId"),
+            sessionId=parse_session_id(params),
+            commonTransactionParams=parse_common_transaction_params(params),
+        )

tck/response/file.py

@@ -2,6 +2,8 @@
 
 from dataclasses import dataclass, field
 
+from tck.response.base import StatusOnlyResponse
+
 
 @dataclass
 class CreateFileResponse:
@@ -20,10 +22,17 @@ class GetFileContentsResponse:
 
 @dataclass
 class GetFileInfoResponse:
+    """Response payload for getFileInfo."""
+
     fileId: str | None = None
     size: str | None = None
     expirationTime: str | None = None
     isDeleted: bool | None = None
     keys: list[str] = field(default_factory=list)
     memo: str | None = None
     ledgerId: str | None = None
+
+
+@dataclass
+class DeleteFileResponse(StatusOnlyResponse):
+    """Response payload for deleteFile."""

tests/unit/file_delete_transaction_test.py

@@ -29,12 +29,17 @@ def test_build_transaction_body(mock_account_ids, file_id):
     assert transaction_body.fileDelete.fileID == file_id._to_proto()
 
 
-def test_missing_file_id():
-    """Test that building a transaction without setting FileID raises a ValueError."""
+def test_missing_file_id(mock_account_ids):
+    """Test that building without FileID leaves fileID unset instead of raising."""
+    account_id, _, node_account_id, _, _ = mock_account_ids
     delete_tx = FileDeleteTransaction()
+    delete_tx.set_node_account_ids([node_account_id])
+    delete_tx.operator_account_id = account_id
+
+    transaction_body = delete_tx.build_transaction_body()
 
-    with pytest.raises(ValueError, match="Missing required FileID"):
-        delete_tx.build_transaction_body()
+    assert transaction_body.HasField("fileDelete")
+    assert not transaction_body.fileDelete.HasField("fileID")
 
 
 def test_set_file_id(file_id):