Monilnarang opened a new pull request, #7286: URL: https://github.com/apache/incubator-seata/pull/7286
- [ ] I have registered the PR [changes](../changes). ### Ⅰ. Describe what this PR did **Summary**: - Separated the test `testMsgSerialize` into 2 tests: `testGlobalSessionMsgSerializationAndDeserialization` and `testBranchSessionMsgSerializationAndDeserialization`. **Elaboration**: - The test `testMsgSerialize` is long, complex and have 2 testing flows which are completely independent of each other. - In the PR I have separated those different testing logics into different tests making the new tests simplified & easier to understand and maintain. The new tests would also have increased debugging and issue finding capabilities. ### Ⅱ. Does this pull request fix one issue? <!-- If that, add "fixes #xxx" below in the next line, for example, fixes #97. --> ### Ⅲ. Why don't you add test cases (unit test/integration test)? ### Ⅳ. Describe how to verify it Successful tests run ### Ⅴ. Special notes for reviews -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@seata.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@seata.apache.org For additional commands, e-mail: notifications-h...@seata.apache.org
