Check for error returned in channel msg_tests (backport #5492) (#5527)

* Check for error returned in channel msg_tests (#5492)

* update test for MsgChannelOpenInit

* update test for MsgChannelOpenTry

* Use error as expError.

* update TestMsgChannelOpenAckValidateBasic

* update TestMsgChannelOpenConfirmValidateBasic

* update TestMsgChannelCloseInitValidateBasic and TestMsgChannelCloseConfirmValidateBasic

* update TestMsgRecvPacketValidateBasic

* update TestMsgTimeoutValidateBasic

* update TestMsgTimeoutOnCloseValidateBasic, TestMsgAcknowledgementValidateBasic, TestMsgChannelUpgradeInitValidateBasic, TestMsgChannelUpgradeTryValidateBasic

* update TestMsgChannelUpgradeAckValidateBasic, TestMsgChannelUpgradeConfirmValidateBasic

* update TestMsgChannelUpgradeOpenValidateBasic and TestMsgChannelUpgradeTimeoutValidateBasic

* update TestMsgChannelUpgradeCancelValidateBasic

* review comment

* reorder test cases

* remove unnecessary expPass variables

---------

Co-authored-by: DimitrisJim <d.f.hilliard@gmail.com>
Co-authored-by: Carlos Rodriguez <carlos@interchain.io>
(cherry picked from commit f51d062)

# Conflicts:
#	modules/core/04-channel/types/msgs_test.go

* merge artifacts

---------

Co-authored-by: Lặc <67097720+expertdicer@users.noreply.github.com>
Co-authored-by: Charly <charly@interchain.berlin>