(ws-axiom) branch master updated: Replace exclude() call with if statement for large.xml in TestGetXMLStreamReader

[veithen]

This is an automated email from the ASF dual-hosted git repository.

veithen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ws-axiom.git


The following commit(s) were added to refs/heads/master by this push:
     new ec5a64fd1 Replace exclude() call with if statement for large.xml in 
TestGetXMLStreamReader
ec5a64fd1 is described below

commit ec5a64fd15dc7c90ce767488ab53ed9762e4c3bc
Author: Andreas Veithen-Knowles <andreas.veit...@gmail.com>
AuthorDate: Mon Mar 23 21:19:43 2026 +0000

    Replace exclude() call with if statement for large.xml in 
TestGetXMLStreamReader
---
 .../main/java/org/apache/axiom/ts/om/OMTestSuiteBuilder.java | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git 
a/testing/axiom-testsuite/src/main/java/org/apache/axiom/ts/om/OMTestSuiteBuilder.java
 
b/testing/axiom-testsuite/src/main/java/org/apache/axiom/ts/om/OMTestSuiteBuilder.java
index 8188ca964..ef137da67 100644
--- 
a/testing/axiom-testsuite/src/main/java/org/apache/axiom/ts/om/OMTestSuiteBuilder.java
+++ 
b/testing/axiom-testsuite/src/main/java/org/apache/axiom/ts/om/OMTestSuiteBuilder.java
@@ -61,10 +61,6 @@ public class OMTestSuiteBuilder extends 
MatrixTestSuiteBuilder {
 
     public OMTestSuiteBuilder(OMMetaFactory metaFactory) {
         this.metaFactory = metaFactory;
-        // This test is particularly slow because it checks all 
XMLStreamReader methods on
-        // every event and many of these invocations throw exceptions. Exclude 
it because
-        // it doesn't add value (with respect to the other 
TestGetXMLStreamReader runs.
-        exclude(org.apache.axiom.ts.om.container.TestGetXMLStreamReader.class, 
"(file=large.xml)");
     }
 
     @Override
@@ -186,8 +182,12 @@ public class OMTestSuiteBuilder extends 
MatrixTestSuiteBuilder {
         addTest(new org.apache.axiom.ts.om.comment.TestSerialize(metaFactory));
         for (XMLSample file : getInstances(XMLSample.class)) {
             for (OMContainerExtractor ce : 
getInstances(OMContainerExtractor.class)) {
-                // TODO: investigate why this causes problems
-                if (!file.getName().equals("character-references.xml")) {
+                // TODO: investigate why character-references.xml causes 
problems
+                // large.xml: this test is particularly slow because it checks 
all XMLStreamReader
+                // methods on every event and many of these invocations throw 
exceptions. It
+                // doesn't add value with respect to the other 
TestGetXMLStreamReader runs.
+                if (!file.getName().equals("character-references.xml")
+                        && !file.getName().equals("large.xml")) {
                     for (BuilderFactory bf : 
getInstances(BuilderFactory.class)) {
                         for (boolean cache : new boolean[] {false, true}) {
                             addTest(