Abacn commented on PR #32909: URL: https://github.com/apache/beam/pull/32909#issuecomment-2436179315
> The Java code that was changed seems like it was more correct and useful. We should perhaps reduce or eliminate use of mockito if it isn't able to cope with pretty basic programming constructs... Totally agree. Current Firestore tests heavily use mockito, in addition to deeply child test classes which is not recommended -- 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]
