Uploaded image for project: 'Lucene - Core'
  1. Lucene - Core
  2. LUCENE-6411

Add AssertingSpans

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 5.2, 6.0
    • None
    • None
    • New

    Description

      Currently Spans needed a lot of asserts in every Spans to make it possibly to debug things. If you miss a single assert, bugs are hidden or not narrowed down the problematic Spans class.

      Instead I think it would be better if we made a test-framework/AssertingSpansQuery that wrapped another SpanQuery and added all the checks. We could use it to really improve the debuggability and coverage of tests.

      Attachments

        Activity

          People

            Unassigned Unassigned
            rcmuir Robert Muir
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: