Merge pull request #210 from f-lopes/fix-compare-assert-assertj-usage

bodewig · web-flow · commit e99d702653c5 · 2020-12-21T09:40:09.000+01:00
Fix assertj usage in CompareAssert helper
diff --git a/xmlunit-assertj3/src/main/java/org/xmlunit/assertj3/CompareAssert.java b/xmlunit-assertj3/src/main/java/org/xmlunit/assertj3/CompareAssert.java
@@ -43,7 +43,7 @@
 import static org.xmlunit.diff.DifferenceEvaluators.chain;
 
 /**
- * Assertion methods for XMLs comparision.
+ * Assertion methods for XMLs comparison.
  *
  * <p><b>Simple Example</b></p>
  *
@@ -388,7 +388,7 @@ private void failComparison(final String type, final String controlSystemId,
             description, expected, actual)
                 .replace("%", "%%"); // any remaining '%' signs should be escaped because assertj tries to format this as well.
 
-        failWithActualExpectedAndMessage(expected, actual, msg);
+        failWithActualExpectedAndMessage(actual, expected, msg);
     }
 
 }

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@`
`43`	`43`	`import static org.xmlunit.diff.DifferenceEvaluators.chain;`
`44`	`44`
`45`	`45`	`/**`
`46`		`- * Assertion methods for XMLs comparision.`
	`46`	`+ * Assertion methods for XMLs comparison.`
`47`	`47`	`*`
`48`	`48`	`* <p><b>Simple Example</b></p>`
`49`	`49`	`*`
`@@ -388,7 +388,7 @@ private void failComparison(final String type, final String controlSystemId,`
`388`	`388`	`description, expected, actual)`
`389`	`389`	`.replace("%", "%%"); // any remaining '%' signs should be escaped because assertj tries to format this as well.`
`390`	`390`
`391`		`- failWithActualExpectedAndMessage(expected, actual, msg);`
	`391`	`+ failWithActualExpectedAndMessage(actual, expected, msg);`
`392`	`392`	`}`
`393`	`393`
`394`	`394`	`}`