[JCR-3408] Query tests should avoid casting to QueryManagerImpl

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.7.1
• Assignee: Alex Deparvu
• Reporter: Alex Deparvu
• Created: 09/Aug/12 09:08
• Updated: 16/Sep/13 07:59
• Resolved: 10/Sep/13 14:38

Description

A lot of query tests flush the search index before to ensure consistency.

Getting the SearchIndex instance happens via casting the QueryManager to the jackrabbit QueryManagerImpl:
((QueryManagerImpl) qm)

I'd like to change this direct casting into a instanceof verification.

This doesn't fix everything but at least it will allow other projects (like oak) to run most of the query tests, without running into ClassCastException(s).