[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:
Issue split
split to CDRIVER-4239 Remove srvMaxHosts tests expecting an... Backlog
split to CSHARP-3992 Remove srvMaxHosts tests expecting an... Backlog
split to CXX-2421 Remove srvMaxHosts tests expecting an... Backlog
split to GODRIVER-2253 Remove srvMaxHosts tests expecting an... Closed
split to MOTOR-864 Remove srvMaxHosts tests expecting an... Closed
split to NODE-3812 Remove srvMaxHosts tests expecting an... Closed
split to PHPC-2016 Remove srvMaxHosts tests expecting an... Closed
split to PYTHON-3040 Remove srvMaxHosts tests expecting an... Closed
split to RUBY-2851 Remove srvMaxHosts tests expecting an... Closed
split to RUST-1126 Remove srvMaxHosts tests expecting an... Closed
split to JAVA-4425 Remove srvMaxHosts tests expecting an... Closed
Related
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:
Key Status/Resolution FixVersion
CDRIVER-4239 Backlog
CXX-2421 Backlog
CSHARP-3992 Backlog
GODRIVER-2253 Fixed 1.12.1
JAVA-4425 Fixed 4.5.0
NODE-3812 Fixed 4.9.0
PHPC-2016 Won't Do
PYTHON-3040 Fixed 4.1
MOTOR-864 Duplicate
RUBY-2851 Fixed 2.18.0
RUST-1126 Duplicate
SWIFT-1433 Duplicate

 Description   

Summary

mongodb/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.

Motivation

Who 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.
Branch: master
https://github.com/mongodb/specifications/commit/8a88d16b252030a0ebb8872ece4c5772ea436eaa

Comment by Jeremy Mikola [ 10/Dec/21 ]

https://github.com/mongodb/specifications/pull/1109

Generated at Thu Feb 08 08:24:29 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.