Move duplicated getIssueFee & getMintFee helpers to CoreumChain in integr-tests

[ysv]

Description

This PR replaces duplicated getIssueFee helper func in multiple locations with a single QueryAssetFTParams method inside CoreumChain struct.
Same refactoring applied to asset/nft params to avoid same issue in the future.
The idea is to avoid code duplication & have a single universal method to query all params for asset/ft & asset/nft modules.

This change is 

[dzmitryhil]

Reviewable status: 0 of 9 files reviewed, 1 unresolved discussion (waiting on @miladz68, @vertex451, @wojtek-coreum, and @ysv)

---

integration-tests/coreum.go line 186 at r1 (raw file):

// QueryAssetFTParams queries for asset/ft module params and returns them.
func (c CoreumChain) QueryAssetFTParams(ctx context.Context, t *testing.T) assetfttypes.Params {
	queryClient := assetfttypes.NewQueryClient(c.ClientContext)

You need to add t.Testing() for that func and text func, if you pass the t *testing.T as arg. This is good practice since helps the testing framework separate test and helper functions and in case of logs or errors, show proper line in the test.

[miladz68]

Reviewed 9 of 9 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @vertex451, @wojtek-coreum, and @ysv)

[ysv]

Reviewable status: 8 of 9 files reviewed, 1 unresolved discussion (waiting on @dzmitryhil, @miladz68, @vertex451, and @wojtek-coreum)

---

integration-tests/coreum.go line 186 at r1 (raw file):

Previously, dzmitryhil (Dzmitry Hil) wrote…

You need to add t.Testing() for that func and text func, if you pass the t *testing.T as arg. This is good practice since helps the testing framework separate test and helper functions and in case of logs or errors, show proper line in the test.

I guess you meant to add t.Helper()

Done

[dzmitryhil]

Reviewable status: 8 of 9 files reviewed, 1 unresolved discussion (waiting on @miladz68, @vertex451, @wojtek-coreum, and @ysv)

---

integration-tests/coreum.go line 186 at r1 (raw file):

Previously, ysv (Yaroslav Savchuk) wrote…

I guess you meant to add t.Helper()

Done

Right.

[dzmitryhil]

Reviewable status: 8 of 9 files reviewed, all discussions resolved (waiting on @miladz68, @vertex451, and @wojtek-coreum)

[miladz68]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @vertex451 and @wojtek-coreum)