feat(tck): Impl Tck endpoint for createAccount (hiero-ledger#1978)

Signed-off-by: Manish Dait <daitmanish88@gmail.com>

Author: manishdait

.codacy.yml

@@ -1,4 +1,7 @@
 ---
+exclude_paths:
+  - "tck/**"
+
 engines:
   bandit:
     enabled: true

.github/workflows/tck-test.yml

@@ -0,0 +1,52 @@
+name: TCK Unit Test
+
+on:
+  push:
+    branches:
+      - "main"
+    paths:
+      - "tck/**"
+      - "tests/tck/**"
+  pull_request:
+    paths:
+      - "tck/**"
+      - "tests/tck/**"
+  workflow_dispatch: {}
+
+permissions:
+  contents: read
+
+
+jobs:
+  tck-unit-test:
+    name: Tck Unit Test
+    runs-on: hl-sdk-py-lin-md
+
+    steps:
+      - name: Harden the runner (Audit all outbound calls)
+        uses: step-security/harden-runner@58077d3c7e43986b6b15fba718e8ea69e387dfcc # v2.15.1
+        with:
+          egress-policy: audit
+      
+      - name: Checkout repository
+        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+
+      - name: Set up Python
+        uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405 # v6.2.0
+        with:
+          python-version: "3.14"
+
+      - name: Install uv
+        uses: astral-sh/setup-uv@e06108dd0aef18192324c70427afc47652e63a82 # v7.5.0
+        with:
+          enable-cache: true
+
+      - name: Install dependencies
+        run: uv sync --all-extras --dev
+
+      - name: Generate Proto Files
+        run: uv run generate_proto.py
+      
+      - name: Run unit tests
+        run: |
+          uv run pytest tests/tck -v

CHANGELOG.md

@@ -9,12 +9,12 @@ This changelog is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.
 ### Src
 - Fix the TransactionGetReceiptQuery to raise ReceiptStatusError for the non-retryable and non success receipt status
 - Refactor `AccountInfo` to use the existing `StakingInfo` wrapper class instead of flattened staking fields. Access is now via `info.staking_info.staked_account_id`, `info.staking_info.staked_node_id`, and `info.staking_info.decline_reward`. The old flat accessors (`info.staked_account_id`, `info.staked_node_id`, `info.decline_staking_reward`) are still available as deprecated properties and will emit a `DeprecationWarning`. (#1366)
-
+- Added abstract `Key` supper class to handle various proto Keys.
 
 ### Examples
 
 ### Tests
-
+- Added TCK endpoint for the createAccount method
 
 ### Docs
 

codecov.yml

@@ -10,5 +10,8 @@ coverage:
         target: 92%
         threshold: 2%
 
+ignore: 
+- "tck/**"
+
 comment:
   layout: "reach, diff, flags"

src/hiero_sdk_python/account/account_create_transaction.py

@@ -2,23 +2,23 @@
 AccountCreateTransaction class.
 """
 
+import ctypes
 from typing import Optional, Union
 import warnings
 
 from hiero_sdk_python.account.account_id import AccountId
 from hiero_sdk_python.channels import _Channel
 from hiero_sdk_python.crypto.evm_address import EvmAddress
-from hiero_sdk_python.crypto.public_key import PublicKey
+from hiero_sdk_python.crypto.key import Key
 from hiero_sdk_python.crypto.private_key import PrivateKey
 from hiero_sdk_python.Duration import Duration
 from hiero_sdk_python.executable import _Method
-from hiero_sdk_python.hapi.services import crypto_create_pb2, duration_pb2, transaction_pb2, basic_types_pb2
+from hiero_sdk_python.hapi.services import crypto_create_pb2, duration_pb2, transaction_pb2
 from hiero_sdk_python.hapi.services.schedulable_transaction_body_pb2 import (
     SchedulableTransactionBody,
 )
 from hiero_sdk_python.hbar import Hbar
 from hiero_sdk_python.transaction.transaction import Transaction
-from hiero_sdk_python.utils.key_utils import Key, key_to_proto
 
 AUTO_RENEW_PERIOD = Duration(7890000)  # around 90 days in seconds
 DEFAULT_TRANSACTION_FEE = Hbar(3).to_tinybars()  # 3 Hbars
@@ -258,12 +258,12 @@ def set_staked_account_id(
         """
         self._require_not_frozen()
         if isinstance(account_id, str):
-            self.staked_account_id = AccountId.from_string(account_id)
-        elif isinstance(account_id, AccountId):
-            self.staked_account_id = account_id
-        else:
+            account_id = AccountId.from_string(account_id)
+        elif not isinstance(account_id, AccountId):
             raise TypeError("account_id must be of type str or AccountId")
-
+        
+        self.staked_account_id = account_id
+        self.staked_node_id = None
         return self
 
     def set_staked_node_id(self, node_id: int) -> "AccountCreateTransaction":
@@ -281,6 +281,7 @@ def set_staked_node_id(self, node_id: int) -> "AccountCreateTransaction":
             raise TypeError("node_id must be of type int")
 
         self.staked_node_id = node_id
+        self.staked_account_id = None
         return self
 
     def set_decline_staking_reward(
@@ -315,21 +316,22 @@ def _build_proto_body(self) -> crypto_create_pb2.CryptoCreateTransactionBody:
             ValueError: If required fields are missing.
             TypeError: If initial_balance is an invalid type.
         """
-        if not self.key:
-            raise ValueError("Key must be set before building the transaction.")
 
         if isinstance(self.initial_balance, Hbar):
             initial_balance_tinybars = self.initial_balance.to_tinybars()
         elif isinstance(self.initial_balance, int):
             initial_balance_tinybars = self.initial_balance
         else:
             raise TypeError("initial_balance must be Hbar or int (tinybars).")
-
-        proto_key = key_to_proto(self.key)
+        
+        # Check for overflow
+        if initial_balance_tinybars >= (2**64):
+            raise OverflowError(f"Value {initial_balance_tinybars} exceeds 64-bit unsigned integer limit.")
 
         proto_body = crypto_create_pb2.CryptoCreateTransactionBody(
-            key=proto_key,
-            initialBalance=initial_balance_tinybars,
+            key=self.key.to_proto_key() if self.key is not None else None,
+            # triggers an INVALID_INITIAL_BALANCE pre-check error instead of a local error.
+            initialBalance=ctypes.c_uint64(initial_balance_tinybars).value,
             receiverSigRequired=self.receiver_signature_required,
             autoRenewPeriod=duration_pb2.Duration(seconds=self.auto_renew_period.seconds),
             memo=self.account_memo,
@@ -338,9 +340,12 @@ def _build_proto_body(self) -> crypto_create_pb2.CryptoCreateTransactionBody:
             decline_reward=self.decline_staking_reward
         )
 
-        if self.staked_account_id:
+        if self.staked_node_id is not None and self.staked_account_id is not None:
+            raise ValueError("Specify either staked_node_id or staked_account_id, not both.")
+
+        if self.staked_account_id is not None:
             proto_body.staked_account_id.CopyFrom(self.staked_account_id._to_proto())
-        elif self.staked_node_id:
+        elif self.staked_node_id is not None:
             proto_body.staked_node_id = self.staked_node_id
 
         return proto_body

src/hiero_sdk_python/contract/contract_id.py

@@ -10,6 +10,7 @@
 from typing import TYPE_CHECKING, Optional
 
 from hiero_sdk_python.crypto.evm_address import EvmAddress
+from hiero_sdk_python.crypto.key import Key
 from hiero_sdk_python.hapi.services import basic_types_pb2
 from hiero_sdk_python.utils.entity_id_helper import (
     parse_from_string,
@@ -26,7 +27,7 @@
 
 
 @dataclass(frozen=True)
-class ContractId:
+class ContractId(Key):
     """
     Represents a unique contract ID on the Hedera network.
 
@@ -90,6 +91,15 @@ def _to_proto(self):
             contractNum=self.contract,
             evm_address=self.evm_address,
         )
+    
+    def to_proto_key(self) -> basic_types_pb2.Key:
+        """
+        Convert the ContractId instance to a protobuf Key object.
+
+        Returns:
+            basic_types_pb2.Key: The protobuf object of Key
+        """
+        return basic_types_pb2.Key(contractID=self._to_proto())
 
     @classmethod
     def from_string(cls, contract_id_str: str) -> "ContractId":

src/hiero_sdk_python/contract/delegate_contract_id.py

@@ -0,0 +1,81 @@
+from dataclasses import dataclass
+from typing import TYPE_CHECKING
+
+from hiero_sdk_python.contract.contract_id import ContractId
+from hiero_sdk_python.hapi.services import basic_types_pb2
+from hiero_sdk_python.utils.entity_id_helper import format_to_string_with_checksum
+
+if TYPE_CHECKING:
+  from hiero_sdk_python.client.client import Client
+
+@dataclass(frozen=True)
+class DelegateContractId(ContractId):
+  """
+  Represents a delegatable contract identifier used as a key in the Hiero network.
+
+  A DelegateContractId is a permissive key type that designates a smart contract 
+  authorized to sign a transaction if it is the recipient of the active message 
+  frame. Unlike a standard ContractID, this key type does not require the code 
+  executing in the current frame to belong to the specified contract.
+  """
+
+  def to_proto_key(self) -> basic_types_pb2.Key:
+    return basic_types_pb2.Key(delegatable_contract_id=self._to_proto())
+  
+  
+
+  def __str__(self) -> str:
+    """
+    Returns the string representation of the DelegateContractId.
+    
+    Format will be 'shard.realm.contract' or 'shard.realm.evm_address_hex'
+    if evm_address is set. Does not include a checksum.
+
+    Returns:
+      str: The string representation of the ContractId.
+    """
+    if self.evm_address is not None:
+      return f"{self.shard}.{self.realm}.{self.evm_address.hex()}"
+
+    return f"{self.shard}.{self.realm}.{self.contract}"
+
+  def __repr__(self) -> str:
+    """
+    Returns a detailed string representation of the ContractId for debugging.
+
+    Returns:
+      str: DelegateContractId(shard=X, realm=Y, contract=Z) or
+      DelegateContractId(shard=X, realm=Y, evm_address=...) if evm_address is set.
+    """
+    if self.evm_address is not None:
+      return f"DelegateContractId(shard={self.shard}, realm={self.realm}, evm_address={self.evm_address.hex()})"
+
+    return f"DelegateContractId(shard={self.shard}, realm={self.realm}, contract={self.contract})"
+
+
+  def to_string_with_checksum(self, client: "Client") -> str:
+    """
+    Generates a string representation with a network-specific checksum.
+
+    Format: 'shard.realm.contract-checksum' (e.g., "0.0.123-vfmkw").
+
+    Args:
+      client (Client): The client instance used to generate the
+        network-specific checksum.
+
+    Returns:
+      str: The string representation with checksum.
+
+    Raises:
+      ValueError: If the DelegateContractId has an `evm_address` set,
+                as checksums cannot be applied to EVM addresses.
+    """
+    if self.evm_address is not None:
+      raise ValueError(
+        "to_string_with_checksum cannot be applied to DelegateContractId with evm_address"
+      )
+
+    return format_to_string_with_checksum(
+      self.shard, self.realm, self.contract, client
+    )
+

src/hiero_sdk_python/crypto/evm_address.py

@@ -1,4 +1,7 @@
-class EvmAddress:
+from hiero_sdk_python.crypto.key import Key
+
+
+class EvmAddress(Key):
     """
     Represents a 20-byte EVM address derived from the rightmost 20 bytes of 
     32 byte Keccak-256 hash of an ECDSA public key.
@@ -34,6 +37,10 @@ def from_string(cls, evm_address: str) -> "EvmAddress":
     def from_bytes(cls, address_bytes: "bytes") -> "EvmAddress":
         """Create an EvmAddress from raw bytes."""
         return cls(address_bytes)
+    
+
+    def to_proto_key(self):
+        raise RuntimeError("to_proto_key() not implemented for EvmAddress")
 
     def to_string(self) -> str:
         """Return the EVM address as a hex string"""