Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐛 Update the integration tests #464

Merged
merged 1 commit into from
Mar 5, 2025

Conversation

lrazovic
Copy link
Member

@lrazovic lrazovic commented Mar 3, 2025

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] [2] [3] [4] [5] [6]
  • Removed unused import ETH_ADDRESS from polimec.ts, BridgeToPolimec.ts, and PolimecToHub.ts. [1] [2] [3]

Documentation Update:

  • Added prerequisites and usage instructions to the README.md file in the chopsticks directory.

@lrazovic lrazovic changed the title fix: update the integration tests 🐛 Update the integration tests Mar 4, 2025
@lrazovic lrazovic marked this pull request as ready for review March 4, 2025 09:18
Copy link

graphite-app bot commented Mar 4, 2025

Graphite Automations

"Auto-assign PRs to author" took an action on this PR • (03/04/25)

1 assignee was added to this PR based on Juan Ignacio Rios's automation.

@lrazovic lrazovic requested a review from dastansam March 4, 2025 09:24
@lrazovic lrazovic force-pushed the 02-27-feat_runtime_api_to_retrieve_evaluation_participation_made_by_a_user branch from abd6341 to bf8b889 Compare March 4, 2025 12:23
@lrazovic lrazovic force-pushed the 03-03-fix_update_the_integration_tests branch 2 times, most recently from 7f21caa to 484326d Compare March 4, 2025 12:35
Copy link
Member Author

lrazovic commented Mar 5, 2025

Merge activity

  • Mar 5, 4:29 AM EST: A user started a stack merge that includes this pull request via Graphite.
  • Mar 5, 4:30 AM EST: Graphite couldn't merge this pull request because a downstack PR 🔥 Funding Rounds Simplification #436 failed to merge.
  • Mar 5, 4:34 AM EST: A user started a stack merge that includes this pull request via Graphite.
  • Mar 5, 5:10 AM EST: Graphite rebased this pull request as part of a merge.
  • Mar 5, 5:11 AM EST: A user merged this pull request with Graphite.

@lrazovic lrazovic changed the base branch from 02-27-feat_runtime_api_to_retrieve_evaluation_participation_made_by_a_user to graphite-base/464 March 5, 2025 10:06
@lrazovic lrazovic changed the base branch from graphite-base/464 to main March 5, 2025 10:08
@lrazovic lrazovic force-pushed the 03-03-fix_update_the_integration_tests branch from 484326d to b080d94 Compare March 5, 2025 10:09
@lrazovic lrazovic merged commit 22f81ae into main Mar 5, 2025
1 check passed
@lrazovic lrazovic deleted the 03-03-fix_update_the_integration_tests branch March 5, 2025 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants