fix: check denom of SpendLimit

[czarcas7ic]

Closes: osmosis-labs/osmosis#2170

What is the purpose of the change

This PR fixes a small bug in the Authz simulator code found when running extended simulations.

Brief Changelog

• Checks that the coins being sent through Authz are a subset of the denoms allotted through the SpendLimit

Testing and Verifying

This change is already covered by existing tests

Documentation and Release Note

• Does this pull request introduce a new feature or user-facing behavior changes? no
• Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? no
• How is the feature or change documented? not applicable

[czarcas7ic]

Once merged I will backport to https://github.com/osmosis-labs/cosmos-sdk/tree/v0.45.0x-osmo-v11 and then update our go.mod on osmosis

Unless we want to wait since its such a small change, but extended sim runs will continue to fail until this is added

[czarcas7ic]

Hmm, I am actually still getting

    simulate.go:263: error on block  274/1000, operation (522/766) from x/authz:
        requested amount is more than spend limit: insufficient funds [cosmos/cosmos-sdk/types/errors/errors.go:273]
        Comment: requested amount is more than spend limit: insufficient funds

with this branch

[czarcas7ic]

I believe we need to actually change this to the following:

if !sendAuth.SpendLimit.IsAllGTE(coins) || !coins.DenomsSubsetOf(sendAuth.SpendLimit) {
	return simtypes.NoOpMsg(authz.ModuleName, TypeMsgExec, "over spend limit"), nil, nil
}

Please lmk if that makes sense

[ValarDragon]

Your suggested change in comment looks correct to me