bug(#300): catch compile

h1alexbel · h1alexbel · commit 6062c33170a5 · 2025-04-15T11:07:52.000+02:00
diff --git a/src/main/java/com/jcabi/xml/XSLDocument.java b/src/main/java/com/jcabi/xml/XSLDocument.java
@@ -441,6 +441,8 @@ private void transformInto(final XML xml, final Result result) {
      */
     private Transformer transformer() {
         final TransformerFactory factory = TransformerFactory.newInstance();
+        final ConsoleErrorListener errors = new ConsoleErrorListener();
+        factory.setErrorListener(errors);
         factory.setURIResolver(this.sources);
         final Transformer trans;
         try {
@@ -459,6 +461,15 @@ private Transformer transformer() {
         for (final Map.Entry<String, Object> ent : this.params.entrySet()) {
             trans.setParameter(ent.getKey(), ent.getValue());
         }
+        if (!errors.summary().isEmpty()) {
+            throw new IllegalArgumentException(
+                String.format(
+                    "Failed to compile stylesheet by %s: %s",
+                    trans.getClass().getName(),
+                    errors.summary()
+                )
+            );
+        }
         return trans;
     }
 
diff --git a/src/test/java/com/jcabi/xml/XSLDocumentTest.java b/src/test/java/com/jcabi/xml/XSLDocumentTest.java
@@ -9,7 +9,6 @@
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
 import java.util.concurrent.TimeUnit;
-import javax.xml.transform.TransformerException;
 import org.apache.commons.lang3.StringUtils;
 import org.hamcrest.MatcherAssert;
 import org.hamcrest.Matchers;
@@ -186,7 +185,7 @@ void catchesXslErrorMessages() {
     }
 
     @Test
-    void catchesXslWarnings() {
+    void catchesXslWarningsDuringCompilation() {
         MatcherAssert.assertThat(
             Assertions.assertThrows(
                 RuntimeException.class,
@@ -196,7 +195,6 @@ void catchesXslWarnings() {
                         .transform(new XMLDocument("<f/>"))
             ).getLocalizedMessage(),
             Matchers.allOf(
-                Matchers.containsString("Warning at xsl:stylesheet on line 31 column 139"),
                 Matchers.containsString(
                     "Stylesheet module file:/second.xsl is included or imported more than once"
                 )
