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

Modify list_all_local_cursors.js test to make failure easier to diagnose.

    • Type: Icon: Improvement Improvement
    • Resolution: Fixed
    • Priority: Icon: Major - P3 Major - P3
    • 4.0.0-rc0
    • Affects Version/s: None
    • Component/s: Querying
    • Labels:
      None
    • Fully Compatible
    • Query 2018-05-07
    • 16

      We have observed the following assertion fail in our continuous integration testing:

      https://github.com/mongodb/mongo/blob/00f32ac53c595f098ea200ab7b9d7278be4a5193/jstests/core/list_all_local_cursors.js#L45

      Specifically, the test failed because the count was 1 instead of 2. The assertion is written in such a way that we can't tell which of the two cursors wasn't found; we also can't tell if the cursor wasn't found at all, or whether one of the cursors wasn't found because the logical session ID did not match. We should rewrite this assertion in order to make these assertions separately, in order to make test failures easier to diagnose.

            Assignee:
            david.storch@mongodb.com David Storch
            Reporter:
            david.storch@mongodb.com David Storch
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: