Fixes #3592. DateField unit tests fail on macos.latest.#3602
Fixes #3592. DateField unit tests fail on macos.latest.#3602tig merged 2 commits intogui-cs:v2_developfrom
Conversation
dodexahedron
left a comment
There was a problem hiding this comment.
This works, but a more formal/appropriate means of achieving platform-specific tests is to use per-platform test methods that have the [SupportedOSPlatform(string)] attribute applied.
For example: [SupportedOSPlatform("macos")]
There is also an inverse attribute available: `[UnsupportedOSPlatform(string)]'
Both can appear multiple times if multiple platforms need to be specified.
|
That can be used in other code, too - not just tests, by the way. |
|
Your suggestion is to only run a unit test that only supports or does not support a certain platform. In this case it is a unit test that runs on all platforms (Windows, Linux and Mac) and there are only constraints in 3 situations on the Mac. I also prefer to use the |
Fixes
DateFieldunit tests fail onmacos.latest#3592Proposed Changes/Todos
Pull Request checklist:
CTRL-K-Dto automatically reformat your files before committing.dotnet testbefore commit///style comments)