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

Add basic LST integration tests #1814

Merged
merged 8 commits into from
Aug 13, 2024
Merged

Add basic LST integration tests #1814

merged 8 commits into from
Aug 13, 2024

Conversation

besated
Copy link
Contributor

@besated besated commented Aug 9, 2024

Describe your changes and provide context

  • forked the steak liquid staking protocol wasm to use for sei integration tests
  • added basic integration tests for bonding and unbonding on associated and unassociated addresses

Missing test cases to add later:

  • harvesting (claiming rewards and reinvesting)
  • verify that the tokens unbonded successfully

Copy link

codecov bot commented Aug 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.83%. Comparing base (fc36b39) to head (a5995ef).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1814      +/-   ##
==========================================
+ Coverage   60.82%   60.83%   +0.01%     
==========================================
  Files         380      380              
  Lines       28021    28021              
==========================================
+ Hits        17043    17046       +3     
+ Misses       9825     9822       -3     
  Partials     1153     1153              

see 3 files with indirect coverage changes

});

describe("Unbonding", async function () {
it("Associated account should be able to unbond", async function () {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feels like this test can be merged in to the associated 'Bonding' test, since it also covers the bonding part.

const { expect } = require("chai");
const { v4: uuidv4 } = require("uuid");

const STEAK_HUB_WASM = "./steak/contracts/steak_hub.wasm";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might need to be ../integration_tests/dapp_tests/steak/contracts/<> for both here

@besated besated merged commit a901610 into main Aug 13, 2024
48 checks passed
@besated besated deleted the bt/app-tests branch August 13, 2024 17:35
yzang2019 added a commit that referenced this pull request Aug 15, 2024
* main:
  Add more DEX dapp tests (#1809)
  Add basic LST integration tests  (#1814)
  Allow CW->ERC pointers to be called through wasmd precompile (#1785)
  Bump nonce even if tx fails (#1778)
  Fix docker setup for local cluster (#1806)
  Tune Configs (#1813)
  V5.7.5 release (#1805)
  Evidence Max Bytes Update (#1812)
  Add dApp Tests (#1802)
  [EVM] Tune configs (#1800)
  Revert dex removal (#1801)
  Do not charge gas for feecollector address query (#1795)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants