Open
Description
What happened?
This library creates a PR that fails to compile:
Ideally, these refactors would only produce changes that compile.
What did you want to happen?
Instead, this code ought to be using the more idiomatic safe-logging:preconditions-assertj
library: https://github.com/palantir/safe-logging/tree/develop/preconditions-assertj/src/main/java/com/palantir/logsafe/testing
Here is the change I manually created:
I don't expect assertj-automation to split the .hasMessage()
into .hasLogMessage()
and .containsArgs()
, but I do think it should have introduced a usage of assertThatLoggableExceptionThrownBy()
here.
Metadata
Metadata
Assignees
Labels
No labels