[DRIVERS-1832] SRV polling spec prose tests should be numbered Created: 02/Jul/21 Updated: 27/Jul/21 |
|
| Status: | Implementing |
| Project: | Drivers |
| Component/s: | SRV Polling |
| Fix Version/s: | None |
| Type: | Spec Change | Priority: | Trivial - P5 |
| Reporter: | Jeremy Mikola | Assignee: | Jeremy Mikola |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||||||||||
| Driver Changes: | Needed | ||||||||||||||||||||||||||||||||||||||||||||||||
| Driver Compliance: |
|
||||||||||||||||||||||||||||||||||||||||||||||||
| Description |
|
Potential driver changes: existing prose tests received no functional changes but are now numbered (1-9). Drivers that label individual tests with the corresponding prose test number can update accordingly. All others may disregard this ticket. Relevant commit: mongodb/specifications@75e10ae SummarySRV polling spec prose tests should be numbered, per Prose test numbering. Motivation
Is this issue urgent?No. Is this ticket required by a downstream team?No. Is this ticket only for tests?Yes. |
| Comments |
| Comment by Githook User [ 27/Jul/21 ] |
|
Author: {'name': 'Jeremy Mikola', 'email': 'jmikola@gmail.com', 'username': 'jmikola'}Message: DRIVERS-1832: Number SRV polling prose tests (#1044) |
| Comment by Jeremy Mikola [ 27/Jul/21 ] |
|
In my experience (libmongoc and PHP), we reference the prose test's number in the test itself (e.g. comment, test name) to make accounting easier. I likely had that in mind when reporting this ticket with "Driver Changes: Needed". If most folks would prefer them removed, I don't feel strongly. In the meantime, I've revised the description to account for the driver changes that may be required. |
| Comment by Jeremy Mikola [ 26/Jul/21 ] |
|
https://github.com/mongodb/specifications/pull/1044 alexander.golin: Why were only CDRIVER and CXX tickets created for this? Did the JIRA script fail midway? |