Make a more ergonomic way of creating simple sdk.Coins

[ValarDragon]

Summary

Often due to compatibility issues, you have to make an SDK.Coins from a single Sdk.Coin, specified by raw parameters. This is especially true within tests. At the moment, the most common ways to do this that I'm aware of are:

myCoins := sdk.Coins{sdk.NewInt64Coin(myDenom, 1000)}
myCoins := sdk.NewCoins(sdk.NewInt64Coin(myDenom, 1000)
myCoins := sdk.NewCoins(sdk.NewCoin(myDenom, sdk.NewInt(1000))

At least in Osmosis, such lines are all over the codebase. They're long, and lots of boilerplate. Can we make a more ergonomic way to do this? Perhaps:

sdk.NewInt64Coins(denoms..., ints...)
sdk.NewInt64Coins(myDenom, 1000)

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged
• Contributor assigned/self-assigned

[alexanderbez]

What's wrong with?

myCoins := sdk.NewCoins(sdk.NewInt64Coin(myDenom, 1000)
myCoins := sdk.NewCoins(sdk.NewCoin(myDenom, sdk.NewInt(1000))

If you need to provide multiple coins then break them onto multiple lines. @ValarDragon you can also create these functions in your application too ;-)

[ValarDragon]

Its long, and variable names make this significantly longer. (And tests take longer time to read/write/review) https://doc.casthighlight.com/alt_longlines100-line-code-shouldnt-long-help-readability/

If folks here disagree, will implement it in our application, but its defn unfortunate to have more compatibility breaks

[alexanderbez]

I personally don't see anything wrong with the above snippet I pasted. But I'd be happy to review PRs for added methods.