RESOLVED FIXED 69118
NRWT should have an option to not exit early because of image failures
https://bugs.webkit.org/show_bug.cgi?id=69118
Summary NRWT should have an option to not exit early because of image failures
Simon Fraser (smfr)
Reported 2011-09-29 18:36:01 PDT
NRWT should never exit early because of image failures. When running on a new OS version, you can have lots of image failures without necessarily having layout failures.
Attachments
Dirk Pranke
Comment 1 2011-09-29 18:56:53 PDT
I'm not sure I agree with this; it depends on whether pixel tests are enabled or not. If they are enabled, it seems like we should exit early (and running on a new/unsupported OS version seems like the exception rather than the rule). You can change the --exit-after-n-failures threshold to some really high number if you want to defeat this, or just turn off pixel tests. We can certainly change the defaults to do something different if we detect we're on the "future" version of the O/S, though. I'm not yet convinced that that's the right thing to do, either, but it would be easier to convince me of this than to change it for the general case.
Simon Fraser (smfr)
Comment 2 2011-09-29 20:17:14 PDT
OK, I retitled :)
Dirk Pranke
Comment 3 2011-09-30 12:14:46 PDT
I guess I'm not sure what you want ... if you know you're on a new release that doesn't have correct images yet, is it good enough to just turn off pixel tests?
Simon Fraser (smfr)
Comment 4 2011-09-30 12:16:27 PDT
No, because I'd still like to see what the pixel diffs look like. Maybe --exit-after-n-failure=10000000 is enough for me. I'd prefer a shorthand for 'never exit' though, and didn't see one in the help output.
Dirk Pranke
Comment 5 2011-09-30 12:19:38 PDT
Ah, I see. Would you want to still want to exit early after N text diffs, and just ignore the pixel failures? Otherwise, as you say, --exit-after-n-failures would work. From a quick glance at the code, I believe that --exit-after-n-failures=0 is equivalent to "never exit". That should probably be documented in the help.
Dirk Pranke
Comment 6 2012-06-08 15:54:53 PDT
smfr ... is --exit-after-n-failures=0 good enough, or would you like image failures to be treated separately?
Dirk Pranke
Comment 7 2012-06-13 19:12:41 PDT
in r120007 we changed the default for --exit-after-n-failures to None (i.e., never exit early) (see bug 73843). Hopefully this is good enough and we don't need to add another flag. Please reopen if there's not the case.
Note You need to log in before you can comment on or make changes to this bug.