Refactor query_settings_index_application_aggregate.js

    • Type: Improvement
    • Resolution: Unresolved
    • Priority: Minor - P4
    • None
    • Affects Version/s: None
    • Component/s: None
    • Query Execution
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None

      Currently the test and the related test library
      query_settings_index_hints_tests.js has the following problems:
       - a lot of code duplication
       - non-transparent assertions (what are we testing?)
       - 'non-atomic' tests testing many things at once (see the previous point)
       
      Ideas for improvement:
       - separate the tests when possible
         * fallback tests
         * query settings index application tests
         * $natural tests
         * query settings + index hints tests
         * ...
       - introduce view dimension as crossProduct([view, non-view], [view, non-view])
       - use the same name for view and non-view to simplify the test methods
       - consider forcing cacheing single-solution plans in classic

            Assignee:
            Unassigned
            Reporter:
            Romans Kasperovics
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: