<div class="gmail_quote">On Wed, Apr 11, 2012 at 4:05 PM, Jacob Goldstein <span dir="ltr"><<a href="mailto:jacobg@adobe.com">jacobg@adobe.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div style="font-size:14px;font-family:Calibri,sans-serif;word-wrap:break-word"><div>Don't people currently fix tests that are failing and that they didn't author?</div></div></blockquote><div> </div><div>We don't fix imported tests if you meant modifying html files, etc... to fix a broken test file.</div>

<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="font-size:14px;font-family:Calibri,sans-serif;word-wrap:break-word"><div>If so, isn't that similar as generating a new ref file in that you first have to understand what is being tested?  There would be the old pixel file to go from, so in many cases this may not be too hard.</div>

</div></blockquote><div><br></div><div>Yes, but determining whether a pixel result is correct or not is easier than determining whether a reference file is correct AND adequately addresses all intended test purposes.</div>

<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="font-size:14px;font-family:Calibri,sans-serif;word-wrap:break-word"><div>On the other hand, having the test author create the ref file is definitely preferable.  What would we do in the case where a test author is no longer available?</div>

</div></blockquote><div><br></div><div>Someone needs to figure out the intended purpose of the test and make a call. I've done quite few of those for editing tests we have (not imported tests); I've converted them to dumpAsText and dump-as-markup tests.</div>

<div><br></div><div>- Ryosuke</div><div><br></div></div>