Refactor/richard review v5

[richard-bachman]

Description

closes: #XXXX

Tasks

• Modify (ms *MatchState) Result() in exchange/types/matching.go
• Add test cases
• Fix some chorse

References

---

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

• Appropriate labels applied
• Targeted PR against correct branch
• Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
• Code follows the module structure standards.
• Wrote unit and integration
• Updated relevant documentation (docs/) or specification (x/<module>/spec/)
• Added relevant godoc comments.
• Re-reviewed Files changed in the Github PR explorer
• Review Codecov Report in the comment section below once CI passes

[richard-bachman]

All required changes are cherry-picked into refactor/v5. So, this PR is closed.