Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-100000

Emit the correct test number for failures due to missing test results

    • Type: Icon: Task Task
    • Resolution: Fixed
    • Priority: Icon: Trivial - P5 Trivial - P5
    • 8.2.0-rc0
    • Affects Version/s: None
    • Component/s: None
    • None
    • Query Optimization
    • Fully Compatible
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      If tests are numbered 10, 20, and 30, then the .results file looks something like

      10
      :sortFull ....
      [
      ...,
      ...,
      ...
      ]
      
      20
      :sortFull ....
      [
      ...,
      ...,
      ...
      ]
      
      30
      :sortFull ....
      [
      ...,
      ...,
      ...
      ]
      

      However, if the .actual file is missing a test (e.g. test #20) due to an execution error (e.g. an operator is no longer supported), then it'll look like

      10
      :sortFull ....
      [
      ...,
      ...,
      ...
      ]
      
      30
      :sortFull ....
      [
      ...,
      ...,
      ...
      ]
      

      Hunk headers will pick up the "latest test number" as 10, when the actual error is in test 20 (by the absence of its results).

      If possible, we should try to make this messaging clearer by detecting this situation and correctly indicating that test 20 is missing, rather than referencing the previous test.

            Assignee:
            daniel.segel@mongodb.com Daniel Segel
            Reporter:
            william.qian@mongodb.com William Qian
            Votes:
            6 Vote for this issue
            Watchers:
            22 Start watching this issue

              Created:
              Updated:
              Resolved:
              None
              None
              None
              None