- Issue created by @spokje
- Merge request !6155Convert OptionsDynamicValuesValidationTest into a Kernel test → (Closed) created by spokje
- Status changed to Needs review
10 months ago 7:44am 14 January 2024 - Status changed to RTBC
10 months ago 4:25pm 16 January 2024 - 🇺🇸United States smustgrave
Conversion seems good, coverage still appears to be there.
- Status changed to Needs review
10 months ago 6:24pm 18 January 2024 - Status changed to RTBC
10 months ago 7:42pm 18 January 2024 - 🇺🇸United States smustgrave
Noted that user IDs shouldn't be random.
Update seems done.
- Status changed to Fixed
10 months ago 9:03pm 18 January 2024 - 🇬🇧United Kingdom catch
Follow-up for the kernel test with the random uids might be good yeah, just couldn't see why it was necessary and makes me wonder what I'm missing.
Committed/pushed to 11.x and cherry-picked to 10.2.x, thanks!
- 🇳🇱Netherlands spokje
Opened 📌 Use random user IDs in tests only only when intended to Active as for the above comment.
Automatically closed - issue fixed for 2 weeks with no activity.