More concise handling of optional migratedFromId in test

This commit is contained in:
Shane Kilkelly 2018-12-05 15:00:00 +00:00
parent 709cb0887c
commit 8cb715014c

View file

@ -83,7 +83,10 @@ public class SnapshotAPIStateBuilder {
String projectName,
JsonObject jsonGetDoc
) {
String migratedFromId = null;
if (jsonGetDoc.has("migratedFromId")) {
migratedFromId = jsonGetDoc.get("migratedFromId").getAsString();
}
getDoc.put(
projectName,
new GetDocResult(
@ -92,21 +95,9 @@ public class SnapshotAPIStateBuilder {
jsonGetDoc.get("createdAt").getAsString(),
jsonGetDoc.get("email").getAsString(),
jsonGetDoc.get("name").getAsString(),
jsonGetDoc.get("migratedFromId").getAsString()
migratedFromId
)
);
} else {
getDoc.put(
projectName,
new GetDocResult(
jsonGetDoc.get("error"),
jsonGetDoc.get("versionID").getAsInt(),
jsonGetDoc.get("createdAt").getAsString(),
jsonGetDoc.get("email").getAsString(),
jsonGetDoc.get("name").getAsString()
)
);
}
}
private void addGetSavedVersForProject(