Small changes to AtlasDelta toDiffViewFriendlyString method (#193)

* Added change reason to diff friendly string * Small change to diff view printout for better viewability * Reverted a test I accidentally changed * Removed uneeded opening newline in diff print. * Changed bracket to brace for more consistency
osmlab · Aug 9, 2018 · c3d93b9 · c3d93b9
1 parent 205907e
commit c3d93b9
Showing 1 changed file with 10 additions and 9 deletions.
diff --git a/src/main/java/org/openstreetmap/atlas/geography/atlas/delta/Diff.java b/src/main/java/org/openstreetmap/atlas/geography/atlas/delta/Diff.java
@@ -69,14 +69,14 @@ public static String toDiffViewFriendlyString(final Iterable<Diff> diffs)
     {
         final String newLine = System.getProperty("line.separator");
         final StringBuilder builder = new StringBuilder();
-        builder.append("[Diffs: ");
+        builder.append("Diffset {");
         final StringList list = new StringList();
         for (final Diff diff : diffs)
         {
             list.add(newLine + diff.toDiffViewFriendlyString());
         }
         builder.append(list.join(newLine));
-        builder.append(newLine + "]");
+        builder.append(newLine + "}");
         return builder.toString();
     }
 
@@ -350,14 +350,15 @@ public String toDiffViewFriendlyString()
     {
         final String newLine = System.getProperty("line.separator");
         final StringBuilder builder = new StringBuilder();
-        builder.append("[Diff: ");
-        builder.append(this.diffType);
+        builder.append("Diff {");
         builder.append(newLine);
-        builder.append("Entity = ");
-        builder.append(this.itemType);
+        builder.append("diffType: " + this.diffType);
         builder.append(newLine);
-        builder.append("ID = ");
-        builder.append(this.identifier);
+        builder.append("diffReason: " + this.diffReason);
+        builder.append(newLine);
+        builder.append("Entity = " + this.itemType);
+        builder.append(newLine);
+        builder.append("ID = " + this.identifier);
         builder.append(newLine);
         if (this.getBaseEntity() != null)
         {
@@ -379,7 +380,7 @@ public String toDiffViewFriendlyString()
             builder.append("null");
         }
         builder.append(newLine);
-        builder.append("]");
+        builder.append("}");
         return builder.toString();
     }