PatchApplier.Result.Error: mark fields final

Fields of an Error instance shouldn't be modifiable after its creation.
Adapt tests which were setting hh to null to skip asserting it.

Change-Id: I0f55c1d5cd529aa510029054e6f05bd2637d1bca

1 files changed, 6 insertions, 8 deletions

diff --git a/org.eclipse.jgit.test/tst/org/eclipse/jgit/patch/PatchApplierTest.java b/org.eclipse.jgit.test/tst/org/eclipse/jgit/patch/PatchApplierTest.java
index eec403a976..5507f8572d 100644
--- a/org.eclipse.jgit.test/tst/org/eclipse/jgit/patch/PatchApplierTest.java
+++ b/org.eclipse.jgit.test/tst/org/eclipse/jgit/patch/PatchApplierTest.java
@@ -373,10 +373,9 @@ public class PatchApplierTest {
 
 			assertEquals(result.getErrors().size(), 1);
 			PatchApplier.Result.Error error = result.getErrors().get(0);
-			error.hh = null; // We don't assert the hunk header as it is a
-								// complex object with lots of internal state.
-			assertEquals(error, new PatchApplier.Result.Error(
-					"cannot apply hunk", "allowconflict", null, true));
+			assertEquals("cannot apply hunk", error.msg);
+			assertEquals("allowconflict", error.oldFileName);
+			assertTrue(error.isGitConflict());
 			verifyChange(result, "allowconflict", true, 1);
 		}
 
@@ -388,10 +387,9 @@ public class PatchApplierTest {
 
 			assertEquals(result.getErrors().size(), 1);
 			PatchApplier.Result.Error error = result.getErrors().get(0);
-			error.hh = null; // We don't assert the hunk header as it is a
-								// complex object with lots of internal state.
-			assertEquals(error, new PatchApplier.Result.Error(
-					"cannot apply hunk", "ConflictOutOfBounds", null, true));
+			assertEquals("cannot apply hunk", error.msg);
+			assertEquals("ConflictOutOfBounds", error.oldFileName);
+			assertTrue(error.isGitConflict());
 			verifyChange(result, "ConflictOutOfBounds", true, 1);
 		}