Fix: replace deprecated pkg_resources.resource_string

Replace usage in chains/ethereum.py by `importlib.resources.load_text`.

src/aleph/chains/ethereum.py

@@ -1,10 +1,10 @@
 import asyncio
 import functools
+import importlib.resources
 import json
 import logging
 from typing import AsyncIterator, Dict, Tuple
 
-import pkg_resources
 from aleph_message.models import Chain
 from configmanager import Config
 from eth_account import Account
@@ -49,15 +49,15 @@ def get_web3(config) -> Web3:
 
 async def get_contract_abi():
     return json.loads(
-        pkg_resources.resource_string(
-            "aleph.chains", "assets/ethereum_sc_abi.json"
-        ).decode("utf-8")
+        importlib.resources.read_text(
+            "aleph.chains.assets", "ethereum_sc_abi.json", encoding="utf-8"
+        )
     )
 
 
 async def get_contract(config, web3: Web3):
     return web3.eth.contract(
-        config.ethereum.sync_contract.value, abi=await get_contract_abi()
+        address=config.ethereum.sync_contract.value, abi=await get_contract_abi()
     )
 
 

tests/chains/test_ethereum.py

@@ -0,0 +1,18 @@
+import pytest
+from configmanager import Config
+from web3 import Web3
+
+from aleph.chains.ethereum import get_contract
+
+
+@pytest.fixture
+def web3():
+    return Web3()
+
+
+@pytest.mark.asyncio
+async def test_get_contract(mock_config: Config, web3: Web3):
+    contract = await get_contract(config=mock_config, web3=web3)
+    # The type hint provided by the web3 library is clearly wrong. This is a simple check
+    # to ensure that we get a proper web3 object. Improve as needed.
+    assert contract.w3 == web3