RFR: 8289508: Improve test coverage for XPath Axes: ancestor, ancestor-or-self, preceding, and preceding-sibling [v4]

Joe Wang joehw at openjdk.org
Thu Sep 15 21:17:46 UTC 2022


On Thu, 15 Sep 2022 06:51:34 GMT, Rahul Prabhu <duke at openjdk.org> wrote:

>> Created two files for testing XPath Axes: XPathAncestors.java tests the ancestor and ancestor-or-self axes, while XPathPrecedingTest.java tests the preceding and preceding-sibling axes.
>
> Rahul Prabhu has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8289508: fixed whitespace

Looks good overall. 

Note that if the editor you use is prone to add whitespaces, running the normalizer before commit would do the trick.

test/jaxp/javax/xml/jaxp/unittest/xpath/XPathAncestorsTest.java line 121:

> 119: 
> 120:     /*
> 121:      * DataProvider: provides XPath expressions that return null

... expressions that return empty NodeSet.

Note that the test called to evaluate to a Node though the expressions themselves return an empty NodeSet.

test/jaxp/javax/xml/jaxp/unittest/xpath/XPathAncestorsTest.java line 124:

> 122:      */
> 123:     @DataProvider(name = "noResults")
> 124:     public Object[][] getNoResults() {

s/noResults/emptyNodeSet. Refer to the above, it's therefore not that it returns "no result" but the processor treats requesting Node on an empty NodeSet as null.

-------------

PR: https://git.openjdk.org/jdk/pull/10211


More information about the core-libs-dev mailing list