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

Relax `.ignore()` requirements for `Status` and `StatusWith< T >` returning functions in `ASSERT_THROWS` type statements.

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.1.4
    • Component/s: None
    • Labels:
      None

      Description

      Although a function which reports its errors by both throwing and bad Status codes is a bad idea, that's no reason to inconvenience those who need to test such functions.  We should clean up those functions, but it shouldn't be necessary to explicitly ignore their status returns when testing their throw behavior – the test is sufficient to indicate that `Status` is ignored.

        Attachments

          Activity

            People

            Assignee:
            adam.martin ADAM Martin (Inactive)
            Reporter:
            adam.martin ADAM Martin (Inactive)
            Participants:
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: