flex-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alex Harui <aha...@adobe.com>
Subject Re: Build failed in Jenkins: flex-sdk_mustella #290
Date Fri, 06 Sep 2013 02:49:23 GMT

On 9/5/13 3:43 PM, "Mark Kessler" <kesslerconsulting@gmail.com> wrote:

>Ok so I ran [1] with createdImages and the test passed.  Deleted the two
>XML files it created and ran [2] without the create option and it still
>passed.  Image looks fine.  So do I upload the two new baseline images ?
Well, what are the differences between the new baseline and the old one?
You pretty much have to check the differences first.  Most tests will
consistently render and compare to a new baseline, but what we care about
is whether that new baseline is exposing a bug.  There could be a massive
rowHeight change, for example.  Use ImageDiffAIR to compare. Or send me
the old and new baseline.  I'll try to run that test now and look at the
diffs if it fails.  I forgot to do that earlier.


>[1] mini_run.sh  -caseName=Editable_variableRowHeight
>[2] mini_run.sh  -caseName=Editable_variableRowHeight
>On Thu, Sep 5, 2013 at 12:41 PM, Alex Harui <aharui@adobe.com> wrote:
>> On 9/5/13 6:47 AM, "Kessler CTR Mark J" <mark.kessler.ctr@usmc.mil>
>> >Not sure how we changed a bitmap for variableRowHeight with the 3 below
>> >commits.  I can look at the bitmaps when I get home.  If it looks
>> >correct, how do I update the bitmap it uses to check with?  Just copy
>> >it's compared one and push it back to the sdk?
>> Recommended practice is to run the test with -createImages option (and
>> then delete the associated png.xml and run it again without to verify
>> there aren't any differences caused by that option).
>> Sometimes there are differences and the test needs upgrading to
>> those differences.  If you just push the "compared one" then someone
>> will be confused later if another change is required.
>> Did this run use a different player?  If so, I wouldn't push a new
>> I would investigate the differences, and if they are just anti-aliasing
>> differences, then I would add the png.xml file generated from the 11.1
>> player if that gets it to pass.

View raw message