[SERVER-58962] Remove require_fcv_51 from api_version_new_50_language_features.js Created: 30/Jul/21 Updated: 13/Dec/21 Resolved: 13/Dec/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Siran Wang (Inactive) | Assignee: | Charlie Swanson |
| Resolution: | Won't Fix | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Sprint: | QO 2021-09-06, QO 2021-09-20, QO 2021-10-04, QO 2021-10-18, QO 2021-11-01, QO 2021-11-15, QO 2021-11-29, QO 2021-12-13, QO 2021-12-27, QO 2021-08-23 | ||||||||||||
| Participants: | |||||||||||||
| Description |
|
When implement retryable_writes through resmoke, we find that error code for the test is not correct. Please take a look on the error test below, expect error code is 168 while the throwed error code is 323. Please either remove the tag "require_fcv_51" or change the function "assertAggregateFailsWithAPIstrict" from "api_version_new_50_language_features.js" in the future. |
| Comments |
| Comment by Charlie Swanson [ 13/Dec/21 ] |
|
This issue has been lingering for a while and is now not that impactful after releasing 5.1 and onto 5.2. The tag needed to be added to test some new language feature that was actually new in 5.1 and not 5.0, so we could have split that out into it's own test to get the extra coverage back for the 5.0 features. Unfortunately this never rose to the top of the todo list, but I think the effort is not worth the fix at this point just to uncover slightly more multi version test coverage. |