Skip to content

Commit

Permalink
Clean up token-bindings-test
Browse files Browse the repository at this point in the history
  • Loading branch information
JakeHartnell committed Sep 3, 2023
1 parent 0cd084b commit 62069ac
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions packages/bindings-test/src/multitest.rs
Original file line number Diff line number Diff line change
Expand Up @@ -61,15 +61,15 @@ impl Module for TokenFactoryModule {
type QueryT = TokenFactoryQuery;
type SudoT = Empty;

// Builds a mock rust implementation of the expected osmosis functionality for testing
// Builds a mock rust implementation of the expected Token Factory functionality for testing
fn execute<ExecC, QueryC>(
&self,
api: &dyn Api,
storage: &mut dyn Storage,
router: &dyn CosmosRouter<ExecC = ExecC, QueryC = QueryC>,
block: &BlockInfo,
sender: Addr,
msg: TokenFactoryMsg,
msg: Self::ExecT,
) -> AnyResult<AppResponse>
where
ExecC: Debug + Clone + PartialEq + JsonSchema + DeserializeOwned + 'static,
Expand All @@ -85,8 +85,6 @@ impl Module for TokenFactoryModule {
}
ADMIN.save(storage, &new_token_denom, &sender)?;

// TODO: charge the creation fee (once params is supported)

let mut denoms = DENOMS_BY_CREATOR
.may_load(storage, &sender)?
.unwrap_or_default();
Expand Down Expand Up @@ -177,7 +175,7 @@ impl Module for TokenFactoryModule {
ExecC: Debug + Clone + PartialEq + JsonSchema + DeserializeOwned + 'static,
QueryC: CustomQuery + DeserializeOwned + 'static,
{
bail!("sudo not implemented for OsmosisModule")
bail!("sudo not implemented for TokenFactoryModule")
}

fn query(
Expand All @@ -186,7 +184,7 @@ impl Module for TokenFactoryModule {
storage: &dyn Storage,
_querier: &dyn Querier,
_block: &BlockInfo,
request: TokenFactoryQuery,
request: Self::QueryT,
) -> anyhow::Result<Binary> {
match request {
TokenFactoryQuery::FullDenom {
Expand Down

0 comments on commit 62069ac

Please sign in to comment.