Debug query visitor test

FoundatioFx / Foundatio.Parsers

A lucene style query parser that is extensible and allows modifying the query.

https://www.nuget.org/packages/Foundatio.Parsers.LuceneQueries/

Apache License 2.0

66 stars 19 forks source link

Debug query visitor test #61

Closed gjactat closed 3 years ago

gjactat commented 3 years ago

This pull requests adds a unit test that ensures that the DebugQueryVisitor reflects the right Term Fields. It also contains a proposed fix on DebugVisitorQuery output (to satisfy the test above).

CLAassistant commented 3 years ago

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Guillaume JACTAT seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

ejsmith commented 3 years ago

This has been resolved by showing both the current field name and the original if there is one for terms and groups. It was already doing that for other node types. Thank you for finding this issue and submitting a PR!