assertj-automation
assertj-automation copied to clipboard
Migrate to safe-logging:preconditions-assertj
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.