Improve test case to show that ignoreErrors works indeed.

Author: uhafner

src/test/java/edu/hm/hafner/coverage/parser/CoberturaParserTest.java

@@ -32,14 +32,13 @@ CoberturaParser createParser() {
     void shouldIgnoreMissingConditionAttribute() {
         Node duplicateMethods = readReport("cobertura-missing-condition-coverage.xml");
 
-        assertThat(duplicateMethods.getAll(FILE)).extracting(Node::getName)
-                .containsExactly("DataSourceProvider.cs");
-        assertThat(duplicateMethods.getAll(CLASS)).extracting(Node::getName)
-                .containsExactly("VisualOn.Data.DataSourceProvider");
-        assertThat(duplicateMethods.getAll(METHOD)).extracting(Node::getName)
-                .containsExactly("Enumerate()");
+        verifySmallTree(duplicateMethods);
         assertThat(getLog().hasErrors()).isFalse();
 
+        verifyBranchCoverageOfLine61(duplicateMethods);
+    }
+
+    private void verifyBranchCoverageOfLine61(final Node duplicateMethods) {
         var file = duplicateMethods.getAllFileNodes().get(0);
         assertThat(file.getCoveredOfLine(61)).isEqualTo(2);
         assertThat(file.getMissedOfLine(61)).isEqualTo(0);
@@ -49,19 +48,24 @@ void shouldIgnoreMissingConditionAttribute() {
     void shouldIgnoreDuplicateMethods() {
         Node duplicateMethods = readReport("cobertura-duplicate-methods.xml", new CoberturaParser(true));
 
+        verifySmallTree(duplicateMethods);
+        assertThat(getLog().hasErrors()).isTrue();
+        assertThat(getLog().getErrorMessages())
+                .contains("Skipping duplicate method 'VisualOn.Data.DataSourceProvider' for class 'Enumerate()'");
+
+        verifyBranchCoverageOfLine61(duplicateMethods);
+
+        assertThatIllegalArgumentException().isThrownBy(
+                () -> readReport("cobertura-duplicate-methods.xml", new CoberturaParser()));
+    }
+
+    private void verifySmallTree(final Node duplicateMethods) {
         assertThat(duplicateMethods.getAll(FILE)).extracting(Node::getName)
                 .containsExactly("DataSourceProvider.cs");
         assertThat(duplicateMethods.getAll(CLASS)).extracting(Node::getName)
                 .containsExactly("VisualOn.Data.DataSourceProvider");
         assertThat(duplicateMethods.getAll(METHOD)).extracting(Node::getName)
                 .containsExactly("Enumerate()");
-        assertThat(getLog().hasErrors()).isTrue();
-        assertThat(getLog().getErrorMessages())
-                .contains("Skipping duplicate method 'VisualOn.Data.DataSourceProvider' for class 'Enumerate()'");
-
-        var file = duplicateMethods.getAllFileNodes().get(0);
-        assertThat(file.getCoveredOfLine(61)).isEqualTo(2);
-        assertThat(file.getMissedOfLine(61)).isEqualTo(0);
     }
 
     @Test