[SERVER-37365] Relax `.ignore()` requirements for `Status` and `StatusWith< T >` returning functions in `ASSERT_THROWS` type statements. Created: 27/Sep/18  Updated: 29/Oct/23  Resolved: 02/Oct/18

Status: Closed
Project: Core Server
Component/s: None
Affects Version/s: None
Fix Version/s: 4.1.4

Type: Improvement Priority: Major - P3
Reporter: ADAM Martin (Inactive) Assignee: ADAM Martin (Inactive)
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Backwards Compatibility: Fully Compatible
Sprint: Service Arch 2018-10-08
Participants:

 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.



 Comments   
Comment by Githook User [ 02/Oct/18 ]

Author:

{'name': 'ADAM David Alan Martin', 'email': 'adam.martin@10gen.com', 'username': 'adamlsd'}

Message: SERVER-37365 Don't use `ignore()` in some tests.

It shouldn't be necessary to explicitly ignore status returns in
functions when testing their throw behavior - the test is sufficient
to indicate that `Status` is ignored.
Branch: master
https://github.com/mongodb/mongo/commit/0380ac1465fcd05e2577351d5df226eb6bbccaa5

Generated at Thu Feb 08 04:45:46 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.