[SOLR-3175] simplify & add test to ensure various query "escape" functions are in sync

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Assignee: Unassigned
• Reporter: Chris M. Hostetter
• Created: 29/Feb/12 04:05
• Updated: 29/Feb/12 04:09

Description

We have three query syntax escape related functions (that i know) of that can't be refactored...

• QueryParser.escape
  • canonical
• ClientUtils.escapeQueryChars
  • part of solrj, doesn't depend directly on QueryParser so that Solr clients on't need the query parser jar locally
• SolrPluginUtils.partialEscape
  • designed to be a negative subset of the full set (ie: all chars except +/-/")

...we should figure out a way to assert in our tests that these are all in agreement (or at least as much as they are ment to be)