elharo opened a new pull request, #114:
URL: https://github.com/apache/xerces-j/pull/114

   Fixes 5 of the 6 commented-out tests in `schema.config` that were listed as 
failing in `build.xml`.
   
   **Root cause analysis:**
   All failures were test bugs, not code bugs:
   
   1. **IgnoreXSIType_C_AC_Test, IgnoreXSIType_C_CA_Test, 
IgnoreXSIType_C_C_Test** — Expected `VALIDATION_NONE` for elements with 
declared attributes (`attr="typeY"`). Attribute validation via the anyType 
wildcard produces `VALIDATION_PARTIAL`. Changed expectations to 
`VALIDATION_PARTIAL`. The `IGNORE_XSI_TYPE` feature itself works correctly.
   
   2. **UseGrammarPoolOnly_True_Test** — Same pattern: expected 
`VALIDATION_NONE` but attribute `attr="typeX"` is declared in the grammar pool 
and validated, producing `VALIDATION_PARTIAL`. Changed expectation to 
`VALIDATION_PARTIAL`. The `USE_GRAMMAR_POOL_ONLY` feature works correctly 
(external schemas are not resolved).
   
   3. **UnparsedEntityCheckingTest** — `testDefaultValid()` had a leftover 
`throw new RuntimeException();` after `checkDefault()` at line 58, causing a 
test error. Removed it.
   
   4. **SurrogatePairLengthTest** — Passes in isolation but fails in shared-JVM 
mode (`forkmode="once"`) because `TypeValidator.java:42` reads the property as 
a `static final` constant at class load time, which happens before the test's 
static initializer runs when other tests load `TypeValidator` first. Left 
commented with an explanation.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to