Esme Povirk (@madewokherd) commented about dlls/comdlg32/tests/itemdlg.c:
+ ok(dwIDCtl == This->button_id, "Expected button ID %lu, got %lu\n", This->button_id, dwIDCtl); + + hr = IFileDialogCustomize_QueryInterface(pfdc, &IID_IServiceProvider, (void**)&psp); + ok(hr == S_OK, "QI for IServiceProvider failed: 0x%08lx\n", hr); + if (FAILED(hr)) + { + return S_OK; + } + + hr = IServiceProvider_QueryService(psp, &IID_IFolderView, &IID_IFolderView2, (void**)&pfv2); + ok(hr == S_OK, "QueryService for IFolderView2 failed: 0x%08lx\n", hr); + if (SUCCEEDED(hr)) + { + psia = NULL; + hr = IFolderView2_GetSelection(pfv2, TRUE, &psia); + todo_wine_if(hr == E_NOTIMPL) `todo_wine_if` should not depend on the result of the thing we're testing. Since Wine is theoretically fixed by the same patch, this todo should be removed.
(If Wine were not fixed by the same patch, we'd want a regular `todo_wine`, to be removed when Wine is fixed.) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10193#note_130976