generally: * we'd prefer when tests are added in a separate commit (in the same MR), marking failing tests with todo_wine so that it's easier to review what's fixed (in the second commit) * we also tend to avoid having code paths depending on a given window version also tests as they are added will fail if run with Wine set with an old version (not that we still test in the CI such an old version) so I wonder if this couldn't be simplified by just removing the parenthesis from the set of characters? (iow, since XP Professional x64 Edition has shipped, eg 21 years ago, all .bat/.cmd files have run on these OSes and subsequent ones have run without the offending ()...) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10797#note_138523