🐛 Update the integration tests (#464)

## What?

This pull request includes several updates and improvements to the `integration-tests/chopsticks` project. The main changes involve updating dependencies, enhancing test cases, and cleaning up the codebase by removing unnecessary console logs and imports.

### Dependency Updates:
* Updated `@acala-network/chopsticks` to version `1.0.3` and `@polkadot-labs/hdkd` to version `0.0.12` in `package.json`.
* Updated `polkadot-api` dependency to version `^1.9.3` in `package.json`.

### Test Case Enhancements:
* Re-enabled previously commented-out test cases for transferring USDC, USDT, and DOT to the Hub in `polimec.test.ts`.

### Code Cleanup:
* Removed unnecessary console logs from various files, including `PolimecManager.ts`, `BaseTransfer.ts`, `BridgeToPolimec.ts`, `HubToPolimec.ts`, `PolimecToHub.ts`, and `PolkadotToPolimec.ts`. [[1]](diffhunk://#diff-17504b68de01739408ba3271f1474bc42e5e8544b68aab412f359b1345b54fa2L77-R78) [[2]](diffhunk://#diff-5dd414351aca9c6251af3425bc64e3032d2da671fd96d6912c104be8e91a2c36L67) [[3]](diffhunk://#diff-d3f932bd8fa0e6df0cbaf55474b048e8008b43a22b2f502d1ea624fd1d0b9b7eL105-L107) [[4]](diffhunk://#diff-1e5c3625dda1821bb7716a768e632713bb3b4bdaeb8105f0e7fb92761aeec0a1R117) [[5]](diffhunk://#diff-a6b4d0706a66eb8e96a37d085dc9a5b248854854c12d437295efddba2ea5908dL49) [[6]](diffhunk://#diff-31316894b2f94a7e3245783d130ae173feb892061bbe6a5fdcacfb445f999004R124-L125)
* Removed unused import `ETH_ADDRESS` from `polimec.ts`, `BridgeToPolimec.ts`, and `PolimecToHub.ts`. [[1]](diffhunk://#diff-b166cb800d57cb015c4fe852e382234955e8733273ff1d7352aa25bf45c81e79L1-R1) [[2]](diffhunk://#diff-d3f932bd8fa0e6df0cbaf55474b048e8008b43a22b2f502d1ea624fd1d0b9b7eL2-R2) [[3]](diffhunk://#diff-a6b4d0706a66eb8e96a37d085dc9a5b248854854c12d437295efddba2ea5908dL2)

### Documentation Update:
* Added prerequisites and usage instructions to the `README.md` file in the `chopsticks` directory.

Author: lrazovic

integration-tests/chopsticks/README.md

@@ -1,37 +1,48 @@
-# chopsticks
+# XCM Integration Tests: Polkadot API x Chopsticks 
+
+## Prerequisites
+
+- [Bun](https://bun.sh/docs/installation)
+
+## Usage
 
 To install dependencies:
 
 ```bash
 bun install
 ```
 
+To generate the Chains descriptors:
+
 ```bash
 bun papi
 ```
 
 > [!NOTE]
-> Sometimes you need to regenerate the Polimec descriptors. To do that, run:
+> If you need to regenerate the Polimec descriptors (e.g. you changed something on the Runtime). You can run:
 >
 > ```bash
+> bun papi
 > bun papi add polimec --wasm ../../target/release/wbuild/polimec-runtime/polimec_runtime.compact.compressed.wasm
 > ```
 
-To start the chains:
-
-```bash
-bun run dev
-```
+> [!NOTE]
+> If you need to regenerate the descriptors. You can delete the `.papi` folder and run:
+> ```bash
+> bun papi add polimec --wasm ../../target/release/wbuild/polimec-runtime/polimec_runtime.compact.compressed.wasm
+> bun papi add bridge -w wss://sys.ibp.network/bridgehub-polkadot
+> bun papi add pah -w wss://sys.ibp.network/statemint
+> bun papi add polkadot -w wss://rpc.ibp.network/polkadot
+> ```
 
-To run the tests:
+To run all the tests:
 
 ```bash
 bun run test
 ```
 
+To run a specific test case, e.g Polkadot to Polimec:
 
-> [!IMPORTANT]
-> TODO: Add:
-> - [ ] Polimec SA on AH: Add ETH balance to it in the Chopstick ovveride
-> - [ ] Polimec to Asset Hub: ETH transfer. This is a "normal" transfer_asset call.
-> - [ ] Polimec to Ethereum: ETH transfer. This is a bit more complex, example extrinsic: https://polkadot.js.org/apps/?rpc=wss%3A%2F%2Fhydration.ibp.network#/extrinsics/decode/0x6b0d04010100a10f040801000007464a69c7e002020907040300c02aaa39b223fe8d0a0e5c4f27ead9083c756cc200130000e8890423c78a0204010002040816040d01000001010088ca48e3e1d0f1c50bd6b504e1312d21f5bd45ed147e3c30c77eb5e4d63bdc6310010102020907040300c02aaa39b223fe8d0a0e5c4f27ead9083c756cc2000201090704081300010300c02aaa39b223fe8d0a0e5c4f27ead9083c756cc20004000d010204000103001501c1413e4178c38567ada8945a80351f7b849600
+```bash
+bun run test polkadot
+```

integration-tests/chopsticks/bun.lock

@@ -1,4 +1,4 @@
-import { INITIAL_BALANCES, ETH_ADDRESS } from '@/constants';
+import { INITIAL_BALANCES } from '@/constants';
 import { Accounts } from '@/types';
 
 export const POLIMEC_WASM =

integration-tests/chopsticks/overrides/polimec.ts

@@ -7,16 +7,16 @@
     "test": "env LOG_LEVEL=error bun test"
   },
   "devDependencies": {
-    "@acala-network/chopsticks": "1.0.2",
+    "@acala-network/chopsticks": "1.0.3",
     "@biomejs/biome": "1.9.4",
-    "@polkadot-labs/hdkd": "0.0.11",
+    "@polkadot-labs/hdkd": "0.0.12",
     "@types/bun": "latest"
   },
   "peerDependencies": {
     "typescript": "^5.7.3"
   },
   "dependencies": {
     "@polkadot-api/descriptors": "file:.papi/descriptors",
-    "polkadot-api": "^1.9.0"
+    "polkadot-api": "^1.9.3"
   }
 }

integration-tests/chopsticks/package.json

@@ -74,9 +74,8 @@ export class PolimecManager extends BaseChainManager {
   async getXcmFee() {
     const api = this.getApi(Chains.Polimec);
     const events = await api.event.PolkadotXcm.FeesPaid.pull();
-    console.dir(events, { depth: null });
 
-    return events[0]?.payload.fees?.[0]?.fun?.value ?? 0n;
+    return (events[0]?.payload.fees?.[0]?.fun?.value as bigint) ?? 0n;
   }
 
   async getTransactionFee() {

integration-tests/chopsticks/src/managers/PolimecManager.ts

@@ -100,7 +100,6 @@ export class ChainSetup {
       'wasm-override': POLIMEC_WASM,
       'import-storage': polimec_storage,
       'build-block-mode': BuildBlockMode.Instant,
-      'runtime-log-level': 5,
     });
   }
 

integration-tests/chopsticks/src/setup.ts

@@ -19,35 +19,35 @@ describe('Polimec -> Hub Transfer Tests', () => {
   });
   afterAll(async () => await chainSetup.cleanup());
 
-  // test(
-  //   'Send USDC to Hub',
-  //   () =>
-  //     transferTest.testTransfer({
-  //       account: Accounts.BOB,
-  //       assets: [[Asset.USDC, TRANSFER_AMOUNTS.TOKENS, AssetSourceRelation.Sibling]],
-  //     }),
-  //   { timeout: 25000 },
-  // );
-  //
-  // test(
-  //   'Send USDT to Hub',
-  //   () =>
-  //     transferTest.testTransfer({
-  //       account: Accounts.BOB,
-  //       assets: [[Asset.USDT, TRANSFER_AMOUNTS.TOKENS, AssetSourceRelation.Sibling]],
-  //     }),
-  //   { timeout: 25000 },
-  // );
+  test(
+    'Send USDC to Hub',
+    () =>
+      transferTest.testTransfer({
+        account: Accounts.BOB,
+        assets: [[Asset.USDC, TRANSFER_AMOUNTS.TOKENS, AssetSourceRelation.Sibling]],
+      }),
+    { timeout: 25000 },
+  );
 
-  // test(
-  //   'Send DOT to Hub',
-  //   () =>
-  //     transferTest.testTransfer({
-  //       account: Accounts.BOB,
-  //       assets: [[Asset.DOT, TRANSFER_AMOUNTS.NATIVE, AssetSourceRelation.Parent]],
-  //     }),
-  //   { timeout: 25000 },
-  // );
+  test(
+    'Send USDT to Hub',
+    () =>
+      transferTest.testTransfer({
+        account: Accounts.BOB,
+        assets: [[Asset.USDT, TRANSFER_AMOUNTS.TOKENS, AssetSourceRelation.Sibling]],
+      }),
+    { timeout: 25000 },
+  );
+
+  test(
+    'Send DOT to Hub',
+    () =>
+      transferTest.testTransfer({
+        account: Accounts.BOB,
+        assets: [[Asset.DOT, TRANSFER_AMOUNTS.NATIVE, AssetSourceRelation.Parent]],
+      }),
+    { timeout: 25000 },
+  );
 
   test(
     'Send PLMC to Hub',

integration-tests/chopsticks/src/tests/polimec.test.ts

@@ -64,7 +64,6 @@ export abstract class BaseTransferTest {
   protected async verifyExecution() {
     const events = await this.destManager.getMessageQueueEvents();
 
-    console.dir(events, { depth: null });
     expect(events).not.toBeEmpty();
     expect(events).toBeArray();
     expect(events).toHaveLength(1);

integration-tests/chopsticks/src/transfers/BaseTransfer.ts

@@ -1,5 +1,5 @@
 import { expect } from 'bun:test';
-import { DEFAULT_TOPIC, ETH_AMOUNT, FEE_AMOUNT, ETH_ADDRESS } from '@/constants';
+import { DEFAULT_TOPIC, ETH_ADDRESS, ETH_AMOUNT, FEE_AMOUNT } from '@/constants';
 import type { BridgerHubManagaer } from '@/managers/BridgeHubManager';
 import type { PolimecManager } from '@/managers/PolimecManager';
 import type { PolkadotHubManager } from '@/managers/PolkadotHubManager';
@@ -102,9 +102,6 @@ export class BridgeToPolimecTransfer extends BaseTransferTest {
       (event) => event.type === 'ForeignAssets' && event.value.type === 'Issued',
     );
 
-    console.log('Issued Events on Destination: \n');
-    console.dir(issuedEventsOnDest, { depth: null });
-
     // TODO: Check why we have 3 events instead of 2 (ETH + DOT). Curently we have 3 events (ETH + DOT + DOT)
     expect(issuedEventsOnDest.length).toBe(3);
 

integration-tests/chopsticks/src/transfers/BridgeToPolimec.ts

@@ -27,7 +27,7 @@ export class HubToPolimecTransfer extends BaseTransferTest {
     super(sourceManager, destManager);
   }
 
-  async executeTransfer({ account, assets }: TransferOptions) {
+  async executeTransfer({ account, assets, fee_asset_item }: TransferOptions) {
     const [sourceBlock, destBlock] = await Promise.all([
       this.sourceManager.getBlockNumber(),
       this.destManager.getBlockNumber(),
@@ -39,6 +39,7 @@ export class HubToPolimecTransfer extends BaseTransferTest {
       toChain: Chains.Polimec,
       assets: versioned_assets,
       recv: account,
+      fee_asset_item: fee_asset_item ?? 0,
     });
 
     const api = this.sourceManager.getApi(Chains.PolkadotHub);
@@ -113,6 +114,7 @@ export class HubToPolimecTransfer extends BaseTransferTest {
       toChain: Chains.Polimec,
       assets: versioned_assets,
       recv: transferOptions.account,
+      fee_asset_item: transferOptions.fee_asset_item ?? 0,
     });
 
     let remoteFeeAssetId: XcmVersionedAssetId;