Akihiro Sagawa (@sgwaki) commented about dlls/kernel32/tests/path.c:
+ WCHAR short_path[MAX_PATH], long_path[MAX_PATH], ext_short_path[4 + MAX_PATH]; + DWORD short_len, long_len; + + short_path[0] = 0; + short_len = GetShortPathNameW(tempdir, short_path, ARRAY_SIZE(short_path)); + ok(short_len, "GetShortPathNameW failed: %lu\n", GetLastError()); + + lstrcpyW(ext_short_path, prefix); + lstrcatW(ext_short_path, short_path); + lstrcatW(ext_short_path, name); + lstrcatW(ext_short_path, backslash); + lstrcatW(ext_short_path, name); + + SetLastError(0xdeadbeef); + long_len = GetLongPathNameW(ext_short_path, long_path, ARRAY_SIZE(long_path)); + todo_wine The `todo_wine` macro is used for failure tests. So, either (1) use `todo_wine`s in the first commit and add new implementation and remove them in the second commit, or (2) add the implementation with the successful tests. Personally, I prefer option (1) because it makes the TODO locations explicit. Sorry for the insufficient explanation in my previous comment.
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/10192#note_133720