Cleanup 03-connection msg validate basic test

[colin-axner]

Summary

TestNewMsgConnectionOpenConfirm is not very ergonomic. It should be restructured to combine the msgs into the test cases:

testCases := []struct {
		name    string
		msg     *types.MsgConnectionOpenTry
		expPass bool
}{

exactly the same as the other test cases in the same file.

1. put the test case name first
2. move testMsgs into the test case struct
3. set expPass bool last

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged/assigned

[expertdicer]

Can you pls review my changes? Thank you

[crodriguezvega]

Can you pls review my changes? Thank you

Sorry for the late reply, but yes, definitely! Thanks for opening quickly a PR!