Pull apache#2292: Modernize codebase with Java improvements - functionalize DefaultModelProcessor#read

Author: Vincent Potucek

impl/maven-impl/src/test/java/org/apache/maven/impl/model/DefaultModelProcessorTest.java

@@ -35,12 +35,7 @@
 import org.junit.jupiter.api.io.TempDir;
 
 import static org.assertj.core.api.AssertionsForClassTypes.assertThat;
-import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertInstanceOf;
-import static org.junit.jupiter.api.Assertions.assertNotNull;
-import static org.junit.jupiter.api.Assertions.assertNull;
-import static org.junit.jupiter.api.Assertions.assertThrows;
-import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.junit.jupiter.api.Assertions.*;
 import static org.mockito.Mockito.any;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
@@ -233,7 +228,28 @@ void locateExistingPomShouldRejectPomInDifferentDirectory() {
 
 
 
+    @Test
+    void readWithParserThrowingModelParserExceptionShouldAddAsSuppressed() throws Exception {
+        ModelXmlFactory factory = mock(ModelXmlFactory.class);
+        ModelParser parser = mock(ModelParser.class);
+        XmlReaderRequest request = mock(XmlReaderRequest.class);
+        Path pomPath = Path.of("project/pom.xml");
+        when(request.getPath()).thenReturn(pomPath);
+        when(request.isStrict()).thenReturn(true);
+
+        ModelParserException parserException = new ModelParserException("Parser error");
+        when(parser.locateAndParse(any(), any())).thenThrow(parserException);
+
+        Model expectedModel = mock(Model.class);
+        when(factory.read(request)).thenReturn(expectedModel);
+
+        Model result = new DefaultModelProcessor(factory, List.of(parser)).read(request);
+
+        assertSame(expectedModel, result);
+        // In this case, since factory succeeds, no exception is thrown
+    }
 
 
 
+    
 }