Update knownAssets tests for BSC and Polygon token support

[Copilot]

Tests for getKnownErc20Tokens were asserting empty arrays for BSC (chainId 56) and Polygon (chainId 137), but the implementation now returns BNB_ERC20_TOKENS and POLYGON_ERC20_TOKENS for these chains.

Changes

• Replaced obsolete empty array assertions with proper validation tests
• Each test verifies token structure (denom, symbol, name, decimals, contractAddress) and correct chainId

it('should return tokens for BSC (chainId 56)', () => {
  const tokens = getKnownErc20Tokens(56);
  expect(tokens.length).toBeGreaterThan(0);
  tokens.forEach((token) => {
    expect(token.chainId).toBe(56);
    // ... other property validations
  });
});

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[cloudflare-workers-and-pages]

Deploying vidulum-app with    Cloudflare Pages

Latest commit:	92e249b
Status:	✅  Deploy successful!
Preview URL:	https://e6487129.vidulum-app.pages.dev
Branch Preview URL:	https://copilot-sub-pr-64.vidulum-app.pages.dev

View logs

[Copilot]

Pull request overview

Updates the knownAssets test suite to reflect that getKnownErc20Tokens now returns non-empty token lists for BSC (56) and Polygon (137), instead of empty arrays.

Changes:

• Replaced the “non-Ethereum chains return empty array” assertions with chain-specific tests for BSC and Polygon.
• Added per-token structure validation (denom, symbol, name, decimals, contractAddress) and chainId checks for both chains.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The new BSC test only checks that denom is defined, but other ERC20 token lists in this file also validate denom format (e.g. startsWith('erc20:')) and denom/contractAddress consistency via getErc20ContractAddress. Consider adding the same assertions here (and optionally the decimals range checks used in "Token Data Integrity") so BSC tokens get the same integrity coverage as Ethereum mainnet.

[Copilot]

Same as BSC: this Polygon test currently validates only presence/type of fields. To match the existing convention in this file, add denom prefix checks (erc20:) and consider asserting getErc20ContractAddress(token.denom) === token.contractAddress (and decimals bounds) so Polygon tokens are validated beyond being non-empty.

[Copilot]

These two tests duplicate the same per-token structure assertions already used for ETH/Solana in this file. Consider extracting a small helper (e.g., expectValidErc20Token(token, chainId)) to keep the tests DRY and make future additions (more chainIds) less error-prone.