[PATCH 1/4] wininet/tests: Update InternetErrorDlg tests taking recent Windows 10 results as a base.
dmitry at baikal.ru
Mon Aug 30 12:58:17 CDT 2021
Jacek Caban <jacek at codeweavers.com> wrote:
> >>> @@ -1583,7 +1586,7 @@ static void test_InternetErrorDlg(void)
> >>> expected = ERROR_INVALID_PARAMETER;
> >>> res = InternetErrorDlg(hwnd, NULL, i, FLAGS_ERROR_UI_FLAGS_NO_UI, NULL);
> >>> - todo_wine_if( test_flags & FLAG_TODO || i == ERROR_INTERNET_INCORRECT_PASSWORD)
> >>> + todo_wine_if ((test_flags & FLAG_TODO) || i == ERROR_INTERNET_INCORRECT_PASSWORD || i == ERROR_INTERNET_INSERT_CDROM || i == ERROR_INTERNET_CLIENT_AUTH_CERT_NEEDED)
> >> Is there a reason why you don't use FLAG_TODO for those? It may not
> >> match what you need for your new tests, but why are you changing half of
> >> its usage here?
> > Because using FLAG_TODO is fragile due to inconsistent behaviour, and it
> > fails in new or old tests. I tried different approaches in hope to make
> > it more clean and less controvercial regarding FLAG_TODO usage, but finally
> > decided to have it this way. I'm still open to suggestions though, however
> > I haven't found anything better while changing the tests to follow recent
> > Windows behaviour.
> If you remove most of them, then you may remove all of them as well. If
> you remove most of them, you may as well replace the existing mechanism.
> How about something like this:
Thanks for keeping this going, the test and the results look good to me,
though I'd remove the use of broken() inside of an if statement. Are you
planning to submit this patch?
More information about the wine-devel