@sgiehl opened this Pull Request on December 5th 2022 Member

Description:

fixes #20072

Review

@sgiehl commented on December 12th 2022 Member

@bx80 adding a ui test would have been too much effort. As we do not yet have separate UI tests for the UserId feature I would have to adjust the OmniFixture or add a complete new set of tests.
In general I guess we would need to add more tests for edge cases, e.g. minimum and maximum length, special chars, XSS values,... Might be something for a new issue maybe.

This Pull Request was closed on December 12th 2022
Powered by GitHub Issue Mirror