diegovallone commented on issue #2760: URL: https://github.com/apache/pekko/issues/2760#issuecomment-4095145653
@pjfanning thanks for your fast response on this, I really appreciate it. I like Copilot's suggestion. The unit test proves the logic is correct in the mocked scenario. The persistence-typed test proves the logic actually solves the real-world problem, and confirms that the mocked scenario is realistic and even surfaces a corner case (the write-behind) that I wasn't aware of. You have a point that it’s not strictly needed, but I can't complain about having an integration test confirming the fix with non-mocked code. Especially in this situation... I'm really surprised this bug made it this far (all the way from akka, so many years ago). I feel like I'm the first person to actually use this feature, which is honestly a bit scary. -- 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]
