Clean up the OpObserver unittest TestFundamentalOnDeleteOutputs

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Fixed
    • Priority: Major - P3
    • 5.2.0, 5.1.2
    • Affects Version/s: None
    • Component/s: None
    • None
    • Fully Compatible
    • 65
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      In particular, the test claims to exercise passing in different StoreDocOption, but those are not part of the input.

      In theory, that variable was to control whether OplogDeleteEntryArgs.deletedDoc should have a value when calling onDelete. In practice the RetryableOptions is sufficient to express the cases we're interested in. Sharing code between the delete test case inputs and the update test case inputs is probably to blame.

            Assignee:
            Daniel Gottlieb (Inactive)
            Reporter:
            Daniel Gottlieb (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: