add get_token_price and get_nft_holders

Author: fasashen

README.md

@@ -85,9 +85,11 @@ logs = ankr_w3.query.get_logs(
 
 - [`nft.get_nfts`](#get_nfts)
 - [`nft.get_nft_metadata`](#get_nft_metadata)
+- `nft.get_nft_holders`
 - [`token.get_token_holders`](#get_token_holders)
 - [`token.get_token_holders_count_history`](#get_token_holders_count_history)
 - [`token.get_token_holders_count`](#get_token_holders_count)
+- `token.get_token_price`
 - [`token.get_account_balance`](#get_account_balance)
 - [`query.get_logs`](#get_logs)
 - [`query.get_blocks`](#get_blocks)

ankr/advanced_apis.py

@@ -164,6 +164,24 @@ def get_token_holders_count(
             raise APIError("no token holders count found")
         return reply.holder_count_history[0]
 
+    def get_token_price(
+        self,
+        blockchain: types.BlockchainName,
+        contract_address: str,
+        **kwargs: Any,
+    ) -> str:
+        reply = self.provider.call_method(
+            rpc="ankr_getTokenPrice",
+            request=types.GetTokenPriceRequest(
+                blockchain=blockchain,
+                contract_address=contract_address,
+                **kwargs,
+            ),
+            reply_type=types.GetTokenPriceReply,
+        )
+
+        return reply.usd_price
+
     def get_account_balance(
         self,
         wallet_address: str,
@@ -228,6 +246,26 @@ def get_nft_metadata(
             types.GetNFTMetadataReply,
         )
 
+    def get_nft_holders(
+        self,
+        blockchain: types.BlockchainName,
+        contract_address: str,
+        limit: Optional[int] = None,
+        **kwargs: Any,
+    ) -> Iterable[types.Address]:
+        return self.provider.call_method_paginated(
+            "ankr_getNFTHolders",
+            types.GetNFTHoldersRequest(
+                blockchain=blockchain,
+                contract_address=contract_address,
+                **kwargs,
+            ),
+            types.GetNFTHoldersReply,
+            iterable_name="holders",
+            iterable_type=types.Address,
+            limit=limit,
+        )
+
 
 class AnkrAdvancedAPI(AnkrQueryAPI, AnkrTokenAPI, AnkrNFTAPI):
     ...

ankr/providers.py

@@ -11,7 +11,7 @@
 from ankr.exceptions import APIError
 
 TRequest = TypeVar("TRequest", bound=types.RPCModel)
-TReply = TypeVar("TReply", bound=types.RPCModel)
+TReply = TypeVar("TReply")
 TRequestPaginated = TypeVar("TRequestPaginated", bound=types.RPCRequestPaginated)
 TReplyPaginated = TypeVar("TReplyPaginated", bound=types.RPCReplyPaginated)
 

ankr/types.py

@@ -32,6 +32,7 @@ class BlockNumberName(str, enum.Enum):
 BlockchainName = Union[Blockchain, str]
 BlockchainNames = Union[BlockchainName, List[BlockchainName]]
 BlockNumber = Union[int, str, BlockNumberName]
+Address = str
 AddressOrAddresses = Union[str, List[str]]
 Topics = Union[str, List[Union[str, List[str]]]]
 
@@ -117,6 +118,18 @@ class GetNFTMetadataReply(RPCModel):
     attributes: Optional[NftAttributes] = None
 
 
+class GetNFTHoldersRequest(RPCRequestPaginated):
+    blockchain: BlockchainName
+    contract_address: Address
+    page_token: Optional[str] = None
+    page_size: Optional[int] = None
+
+
+class GetNFTHoldersReply(RPCReplyPaginated):
+    holders: List[Address]
+    next_page_token: Optional[str] = None
+
+
 class Balance(RPCModel):
     blockchain: str
     token_name: str
@@ -174,6 +187,17 @@ class GetTokenHoldersCountRequest(RPCRequestPaginated):
     page_size: Optional[int] = None
 
 
+class GetTokenPriceRequest(RPCModel):
+    blockchain: BlockchainName
+    contract_address: str
+
+
+class GetTokenPriceReply(RPCModel):
+    usd_price: str
+    blockchain: BlockchainName
+    contract_address: str
+
+
 class DailyHolderCount(RPCModel):
     holder_count: int
     total_amount: str
@@ -315,7 +339,7 @@ class Config:
     contract_address: Optional[str]
     cumulative_gas_used: str
     gas_used: str
-    logs: List[Log]
+    logs: Optional[List[Log]]
     logs_bloom: str
     transaction_hash: str
     hash: str

tests/test_client.py

@@ -97,6 +97,21 @@ def test_get_nft_metadata() -> None:
     assert reply.attributes.name == "Overleveraged"
 
 
+@pytest.mark.webtest
+def test_get_nft_holders() -> None:
+    client = AnkrAdvancedAPI()
+    holders = list(
+        client.get_nft_holders(
+            blockchain="eth",
+            contract_address="0x4100670ee2f8aef6c47a4ed13c7f246e621228ec",
+            limit=10,
+        )
+    )
+
+    assert holders
+    assert len(holders) == 10
+
+
 @pytest.mark.webtest
 def test_get_transactions() -> None:
     client = AnkrAdvancedAPI()
@@ -179,3 +194,26 @@ def test_get_account_balance() -> None:
 
     assert assets
     assert len(assets) > 0
+
+
+@pytest.mark.webtest
+def test_get_token_price() -> None:
+    client = AnkrAdvancedAPI()
+    price = client.get_token_price(
+        contract_address="0x8290333cef9e6d528dd5618fb97a76f268f3edd4",
+        blockchain="eth",
+    )
+
+    assert price
+    assert float(price) > 0
+
+
+@pytest.mark.webtest
+def test_get_token_price__no_price() -> None:
+    client = AnkrAdvancedAPI()
+    price = client.get_token_price(
+        contract_address="0xeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee",
+        blockchain="eth",
+    )
+
+    assert price == "0"