zydronium/google-nomulus
1
0
Fork 0
mirror of https://github.com/google/nomulus.git synced 2025-05-15 17:07:15 +02:00
Code Issues Projects Releases Packages Wiki Activity

Improve error messages for LogsSubject

Browse source 
Currently, hasLogAtLevelWithMessage gave a very generic "no log at level with
message was found" error on failure. The user didn't know if there were no logs
at all, or if there was a small typo in the log etc.

Using the "regular" Truth assertions gives more informative errors in cleaner code.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=149007204
This commit is contained in:
Ben McIlwain 2017-03-02 08:31:42 -08:00
parent 2e969d6ed1
commit f61f1373df
1 changed files with 34 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

47
javatests/google/registry/testing/LogsSubject.java
View file

@ -16,11 +16,16 @@ package google.registry.testing;
import static com.google.common.truth.Truth.assertAbout; import static com.google.common.truth.Truth.assertAbout;
import com.google.common.collect.ImmutableList;
import com.google.common.testing.TestLogHandler; import com.google.common.testing.TestLogHandler;
import com.google.common.truth.AbstractVerb.DelegatedVerb; import com.google.common.truth.AbstractVerb.DelegatedVerb;
import com.google.common.truth.Correspondence;
import com.google.common.truth.FailureStrategy; import com.google.common.truth.FailureStrategy;
import com.google.common.truth.Subject; import com.google.common.truth.Subject;
import google.registry.testing.TruthChainer.And; import google.registry.testing.TruthChainer.And;
import java.util.List;
import java.util.logging.Handler; import java.util.logging.Handler;
import java.util.logging.Level; import java.util.logging.Level;
import java.util.logging.LogRecord; import java.util.logging.LogRecord;
@ -36,27 +41,43 @@ public class LogsSubject extends Subject<LogsSubject, TestLogHandler> {
super(strategy, subject); super(strategy, subject);
} }
public And<LogsSubject> hasNoLogsAtLevel(Level level) { private static final Correspondence<String, String> CONTAINS_CORRESPONDENCE =
new Correspondence<String, String>() {
@Override
public boolean compare(String actual, String expected) {
return actual.contains(expected);
}
@Override
public String toString() {
return "contains";
}
};
private List<String> getMessagesAtLevel(Level level) {
ImmutableList.Builder<String> builder = ImmutableList.<String>builder();
for (LogRecord log : actual().getStoredLogRecords()) { for (LogRecord log : actual().getStoredLogRecords()) {
if (log.getLevel().equals(level)) { if (log.getLevel().equals(level)) {
failWithRawMessage( builder.add(log.getMessage());
"Not true that there are no logs at level %s. Found <%s>.", level, log.getMessage());
} }
} }
return builder.build();
}
public And<LogsSubject> hasNoLogsAtLevel(Level level) {
check()
.withFailureMessage("Logs at level %s", level)
.that(getMessagesAtLevel(level))
.isEmpty();
return new And<>(this); return new And<>(this);
} }
public And<LogsSubject> hasLogAtLevelWithMessage(Level level, String message) { public And<LogsSubject> hasLogAtLevelWithMessage(Level level, String message) {
boolean found = false; check()
for (LogRecord log : actual().getStoredLogRecords()) { .withFailureMessage("Logs at level %s", level)
if (log.getLevel().equals(level) && log.getMessage().contains(message)) { .that(getMessagesAtLevel(level))
found = true; .comparingElementsUsing(CONTAINS_CORRESPONDENCE)
break; .contains(message);
}
}
if (!found) {
failWithRawMessage("Found no logs at level %s with message %s.", level, message);
}
return new And<>(this); return new And<>(this);
} }