[DRIVERS-2006] Remove srvMaxHosts tests expecting an error for invalid values Created: 10/Dec/21 Updated: 20/May/22 |
|
| Status: | Implementing |
| Project: | Drivers |
| Component/s: | Initial DNS Seedlist Discovery |
| Fix Version/s: | None |
| Type: | Task | Priority: | Unknown |
| Reporter: | Jeremy Mikola | Assignee: | Jeremy Mikola |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | yaml-test | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||
| Driver Changes: | Needed | ||||||||||||||||||||||||||||||||||||||||||||||||||||
| Downstream Changes Summary: | Drivers should sync initial-dns-seedlist-discovery spec tests with mongodb/specifications@8a88d16. Several tests originally introduced in DRIVERS-1519 have been removed. |
||||||||||||||||||||||||||||||||||||||||||||||||||||
| Driver Compliance: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||
| Description |
Summarymongodb/specifications@b508f6d introduced initial-dns-seedlist-discovery spec tests that expect an error when specifying an invalid type or integer value for srvMaxHosts. This contradicts the Connection String spec, which states that a warning should be logged and the parsed value ignored. That same commit also introduced uri-options tests for the same cases, which correctly expect a warning. Unlike uri-options, the initial-dns-seedlist-discovery spec test format has no syntax for asserting warnings (only errors), so we should just remove the incorrect (and redundant) tests. MotivationWho is the affected end user?Drivers. Is this issue urgent?It should be done close to DRIVERS-1519 so drivers can remove any work-arounds they might have added to get the incorrect initial-dns-seedlist-discovery tests passing (unless they were being skipped). Is this ticket required by a downstream team?No. Is this ticket only for tests?Yes. |
| Comments |
| Comment by Githook User [ 13/Dec/21 ] |
|
Author: {'name': 'Jeremy Mikola', 'email': 'jmikola@gmail.com', 'username': 'jmikola'}Message: DRIVERS-2006: Remove srvMaxHosts tests expecting an error for invalid values (#1109) These cases should raise warnings, not errors. Since they are already tested by existing uri-options spec tests, these files can simply be removed. |
| Comment by Jeremy Mikola [ 10/Dec/21 ] |