Hi Doug,

makes sense to me to change the test case. Though the spec is imo not 100% clear about this. But I don't see a reason why an implementation shouldn't be allowed to use a subtype of nt:query.

changed as suggested, svn revision: 293273

thanks for reporting this issue.

regards
 marcel

McComsey, Doug wrote:
I am failing this TCK test: org.apache.jackrabbit.test.api.query.SaveTest.testSave

One of the checks is to insure that the query node is of type nt:query:

assertEquals("Query node is not of type nt:query", ntQuery, queryNode.getPrimaryNodeType().getName());

I would like to suggest that it be:

    assertTrue("Query node is not of type nt:query",
                queryNode.isNodeType(ntQuery));

I fail this test because my node definition is not nt:query but instead has 
nt:query as a supertype. I would pass the suggested test.

Doug McComsey
Computer Associates
[EMAIL PROTECTED]


Reply via email to