nit0906 commented on PR #2700:
URL: https://github.com/apache/jackrabbit-oak/pull/2700#issuecomment-3802861824
> ElasticIndexQueryCommonTest>IndexQueryCommonTest.testMultiValuedPropUpdate
There's this test that fails on the PR. It has a remove property action - so
could be possibly impacted by the change in the PR.
The failing test itself however could be improved a bit imo. The last
assertion
> assertEventually(() -> assertQuery("/jcr:root//*[jcr:contains(@" +
mulValuedProp + ", 'foo')]", "xpath", new ArrayList<>()));
should rather be
assertEventually(() -> assertQuery("/jcr:root//*[jcr:contains(@" +
mulValuedProp + ", 'bar')]", "xpath", new ArrayList<>()));
--
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]
