amirdeljouyi commented on PR #8151: URL: https://github.com/apache/incubator-seata/pull/8151#issuecomment-5131326625
Hi @slievrly, thanks for the detailed review. I added comments clarifying the fixture values, current mutation behavior, and negative timeout. I did not add a read-back assertion because the file-backed mutation branches currently acknowledge completion but do not rewrite the source file. I also checked the existing tests; these cases exercise the factory-delegated paths with concrete assertions. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
