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.

    XMLWordPrintableJSON

Details

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major - P3 Major - P3
    • 4.0.0-rc0
    • None
    • Querying
    • None
    • Fully Compatible
    • Query 2018-05-07
    • 16

    Description

      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.

      Attachments

        Activity

          People

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

            Dates

              Created:
              Updated:
              Resolved: