tMarket.test.ts - part#1 #437
Conversation
| ], | ||
| undefined, | ||
| undefined, | ||
| tcContractAddress, |
There was a problem hiding this comment.
I'm wondering why do you test with Trusted Token instead of CW20? Maybe we should test them separately
There was a problem hiding this comment.
thanks, replaced to CW20
as discussed we could do testing with Trusted token in https://github.com/confio/tgrade-app/blob/main/src/__tests__/integration/in-development/tMarketTrustedCircle.test.ts
| undefined, | ||
| undefined, | ||
| tcContractAddress, | ||
| undefined, |
There was a problem hiding this comment.
You can remove these lines. If the last parameter(s) of a function call is undefined you can omit it
| @@ -3,10 +3,9 @@ import { Bech32 } from "@cosmjs/encoding"; | |||
| import { DirectSecp256k1HdWallet } from "@cosmjs/proto-signing"; | |||
| import { makeCosmoshubPath } from "@cosmjs/stargate"; | |||
| import { config } from "config/network"; | |||
| import { OcContract } from "utils/oversightCommunity"; | |||
There was a problem hiding this comment.
What do you use OcContract for?
There was a problem hiding this comment.
just saw your comment, those changes is not related to tMarket tests, it is just improvements.
There was a problem hiding this comment.
it is created
const oC = await OcCommunity.propose(address, comment, {
add_voting_members: {
voters: [memberToAdd],
},
});
in `oversightCommunityContract.test.ts
This PR about writing first part of tMarket.test.ts tests