Open He-Pin opened 10 months ago
The test succeeds, and I think it's generally intentional that this exception is logged.
Instead of sending it to the console when running the test, though, we should probably capture the log output and verify as part of the test that it is found in there.
GrpcExceptionHandlerSpec test exception assertion will record an error log to the console.
Do we have a way to temporarily set Log off ? @raboof @He-Pin So we can separate test for NullPointerException and RuntimeException, and set Log off
Perhaps we could use LogCapturing
(https://pekko.apache.org/docs/pekko/current/typed/testing-async.html#silence-logging-output-from-tests)