Tests/osmosis adapter

[Kayanski]

This PR aims at testing and adding tests to the Osmosis adapter.

The following PR : AbstractSDK/cw-orchestrator#150 should be merged before the current one can be merged.
Dependencies should be changed in patch : adapters/Cargo.toml

Tasks

• Create an osmosis-test-tube testing implementation for the adapter contract
• Create TESTNET tests for actual live pools (and scraped ans data)

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Pages

Latest commit:	9ac5fde
Status:	✅  Deploy successful!
Preview URL:	https://573ca046.abstract-docs.pages.dev
Branch Preview URL:	https://tests-osmosis-adapter.abstract-docs.pages.dev

View logs

[codecov-commenter]

Codecov Report

Merging #15 (cd76897) into main (a9099c2) will decrease coverage by 6.11%.
The diff coverage is 46.66%.

@@            Coverage Diff             @@
##             main      #15      +/-   ##
==========================================
- Coverage   16.87%   10.76%   -6.11%     
==========================================
  Files         241      113     -128     
  Lines       22056    13689    -8367     
==========================================
- Hits         3721     1474    -2247     
+ Misses      18335    12215    -6120     

Impacted Files	Coverage Δ
adapters/contracts/dex/src/handlers/query.rs	0.00% <0.00%> (ø)
...s/contracts/dex/src/exchanges/exchange_resolver.rs	61.11% <50.00%> (+3.53%)	⬆️
adapters/contracts/dex/src/handlers/execute.rs	31.81% <100.00%> (+3.24%)	⬆️

... and 128 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[CyberHoward]

@Buckram123 could this branch be useful to you for osmosis adapter testing?

[Buckram123]

@Buckram123 could this branch be useful to you for osmosis adapter testing?

I had ultra-small test for osmosis swap:

abstract/modules/contracts/adapters/dex/tests/osmosis_swap.rs

Lines 191 to 206 in 6806edb

	fn swap_native() -> anyhow::Result<()> {
	let (chain, pool_id, dex_adapter, os) = setup_osmosis()?;
	let proxy_addr = os.proxy.addr_str()?;
	dex_adapter.swap(("osmo", 100), "atom", OSMOSIS.to_owned())?;
	// check balances
	let uosmo_balance = chain.query_balance(&proxy_addr, UOSMO)?;
	assert_that!(uosmo_balance.u128()).is_equal_to(9_900);
	let uatom_balance = chain.query_balance(&proxy_addr, UATOM)?;
	assert_that!(uatom_balance.u128()).is_equal_to(89);
	Ok(())
	}
	}

But this one looks much better! Would like to update it and get merged

[Kayanski]

@Buckram123 could this branch be useful to you for osmosis adapter testing?

I had ultra-small test for osmosis swap:

abstract/modules/contracts/adapters/dex/tests/osmosis_swap.rs

Lines 191 to 206 in 6806edb

	fn swap_native() -> anyhow::Result<()> {
	let (chain, pool_id, dex_adapter, os) = setup_osmosis()?;
	let proxy_addr = os.proxy.addr_str()?;
	dex_adapter.swap(("osmo", 100), "atom", OSMOSIS.to_owned())?;
	// check balances
	let uosmo_balance = chain.query_balance(&proxy_addr, UOSMO)?;
	assert_that!(uosmo_balance.u128()).is_equal_to(9_900);
	let uatom_balance = chain.query_balance(&proxy_addr, UATOM)?;
	assert_that!(uatom_balance.u128()).is_equal_to(89);
	Ok(())
	}
	}

But this one looks much better! Would like to update it and get merged

Go ahead obviously :)

[Buckram123]

@Kayanski, this and #115 solves the same issue: that we need proxy address on osmosis execution. But taking two different solution to this(solution in this PR doesn't cover queries yet), so I decided to just merge osmosis.rs file with tests in there: 8052a7e

Are you okay to close this one?

[Kayanski]

@Kayanski, this and #115 solves the same issue: that we need proxy address on osmosis execution. But taking two different solution to this(solution in this PR doesn't cover queries yet), so I decided to just merge osmosis.rs file with tests in there: 8052a7e

Are you okay to close this one?

Yes, I have not worked extensively enough on this PR so go ahead ;) Take the tests with you though yeah

[Buckram123]

Tests moved to #115