[CSHARP-2597] Fix GridFS tests that fail intermittently because they were asserting incorrectly Created: 19/Apr/19 Updated: 28/Oct/23 Resolved: 22/May/19 |
|
| Status: | Closed |
| Project: | C# Driver |
| Component/s: | Testing |
| Affects Version/s: | None |
| Fix Version/s: | 2.8.1 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Vincent Kam (Inactive) | Assignee: | Robert Stam |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
Currently, GridFS tests expect operations to complete within a certain amount of time, t. t is occasionally insufficient and requires us to restart our tests on Evergreen. Increasing t would reduce the amount of time it would take for a patch to go green by reducing the number of times we have to restart tests. |
| Comments |
| Comment by Githook User [ 22/May/19 ] |
|
Author: {'email': 'robert@robertstam.org', 'name': 'rstam', 'username': 'rstam'}Message: |
| Comment by Githook User [ 22/May/19 ] |
|
Author: {'email': 'robert@robertstam.org', 'name': 'rstam', 'username': 'rstam'}Message: |
| Comment by Robert Stam [ 20/May/19 ] |
|
Actually, the intermittent failures were not caused by t being too small, but rather by truncation errors that occurred when converting back and forth between .NET DateTime values and BsonDateTime values; A better fix is to compare the actual and expected values using BsonDateTime values instead of DateTime values. |