#422 XMLDocument.make()

Author: yegor256

src/main/java/com/jcabi/xml/XMLDocument.java

@@ -264,6 +264,81 @@ private XMLDocument(
         this.cache = cache;
     }
 
+    /**
+     * Factory method, to avoid checked exceptions.
+     * @param file The path of the XML
+     * @return XML
+     * @since 0.35.0
+     */
+    @SuppressWarnings("PMD.ProhibitPublicStaticMethods")
+    public static XML make(final File file) {
+        try {
+            return new XMLDocument(file);
+        } catch (final FileNotFoundException ex) {
+            throw new IllegalArgumentException(ex);
+        }
+    }
+
+    /**
+     * Factory method, to avoid checked exceptions.
+     * @param file The path of the XML
+     * @return XML
+     * @since 0.35.0
+     */
+    @SuppressWarnings("PMD.ProhibitPublicStaticMethods")
+    public static XML make(final Path file) {
+        try {
+            return new XMLDocument(file);
+        } catch (final FileNotFoundException ex) {
+            throw new IllegalArgumentException(ex);
+        }
+    }
+
+    /**
+     * Factory method, to avoid checked exceptions.
+     * @param uri The URI of the XML
+     * @return XML
+     * @since 0.35.0
+     */
+    @SuppressWarnings("PMD.ProhibitPublicStaticMethods")
+    public static XML make(final URI uri) {
+        try {
+            return new XMLDocument(uri);
+        } catch (final IOException ex) {
+            throw new IllegalArgumentException(ex);
+        }
+    }
+
+    /**
+     * Factory method, to avoid checked exceptions.
+     * @param url The URL of the XML
+     * @return XML
+     * @since 0.35.0
+     */
+    @SuppressWarnings("PMD.ProhibitPublicStaticMethods")
+    public static XML make(final URL url) {
+        try {
+            return new XMLDocument(url);
+        } catch (final IOException ex) {
+            throw new IllegalArgumentException(ex);
+        }
+    }
+
+    /**
+     * Factory method, to avoid checked exceptions.
+     * @param stream The input stream with the XML
+     * @return XML
+     * @since 0.35.0
+     */
+    @SuppressWarnings("PMD.ProhibitPublicStaticMethods")
+    public static XML make(final InputStream stream) {
+        try {
+            return new XMLDocument(stream);
+        } catch (final IOException ex) {
+            throw new IllegalArgumentException(ex);
+        }
+    }
+
     @Override
     public String toString() {
         return XMLDocument.asString(this.cache);

src/test/java/com/jcabi/xml/XMLDocumentTest.java

@@ -140,11 +140,11 @@ void doesNotFailOnFetchingInMultipleThreadsFromDifferentDocuments() {
 
     @RepeatedTest(60)
     void doesNotFailOnXsdValidationInMultipleThreadsWithTheSameDocumentAndXsd() {
-        final XML xml = new XMLDocument("<root>passesValidXmlThrough</root>");
-        final XML xsd = new XMLDocument(XMLDocumentTest.XSD);
         Assertions.assertDoesNotThrow(
             new Together<>(
-                thread -> xml.validate(xsd)
+                thread -> new XMLDocument("<root>passesValidXmlThrough</root>").validate(
+                    new XMLDocument(XMLDocumentTest.XSD)
+                )
             )::asList,
             "XMLDocument should not fail on validation in multiple threads with the same document and XSD"
         );
@@ -214,7 +214,7 @@ void findsWithXpathWithCustomNamespace() throws Exception {
                 file
             )
         ).value();
-        final XML doc = new XMLDocument(file).registerNs("f", "urn:foo");
+        final XML doc = XMLDocument.make(file).registerNs("f", "urn:foo");
         MatcherAssert.assertThat(
             doc.nodes("/f:a/f:b[.='\u0433!']"),
             Matchers.hasSize(1)
@@ -264,8 +264,8 @@ void printsWithNamespace() {
     }
 
     @Test
-    void retrievesDomNode() throws Exception {
-        final XML doc = new XMLDocument(
+    void retrievesDomNode() {
+        final XML doc = XMLDocument.make(
             this.getClass().getResource("simple.xml")
         );
         MatcherAssert.assertThat(