This also means that we'll need to update this xml when we release a
new version of FlexUnit. Is there a way we could reference the
flexunit build.xml's
"${build.version}-${build.number}-flex_${build.sdk}" variables,
instead of having to maintain these manually?

On 4 January 2016 at 14:31, Mihai Chira <mihai.ch...@gmail.com> wrote:
> I was just about to make the same change in flexunit-tests.xml, nice!
> Were the changes in FLEX_34625_Tests also necessary? In other words,
> was UIImpersonator.testDisplay not an IFocusManagerContainer? That
> would be odd.
>
> On 4 January 2016 at 07:59,  <aha...@apache.org> wrote:
>> Repository: flex-sdk
>> Updated Branches:
>>   refs/heads/develop 5bd868ea7 -> db54a3f34
>>
>>
>> this got these test to pass on my mac
>>
>>
>> Project: http://git-wip-us.apache.org/repos/asf/flex-sdk/repo
>> Commit: http://git-wip-us.apache.org/repos/asf/flex-sdk/commit/db54a3f3
>> Tree: http://git-wip-us.apache.org/repos/asf/flex-sdk/tree/db54a3f3
>> Diff: http://git-wip-us.apache.org/repos/asf/flex-sdk/diff/db54a3f3
>>
>> Branch: refs/heads/develop
>> Commit: db54a3f34fb570557a24847224f50eab4c77bd82
>> Parents: 5bd868e
>> Author: Alex Harui <aha...@apache.org>
>> Authored: Sun Jan 3 22:58:15 2016 -0800
>> Committer: Alex Harui <aha...@apache.org>
>> Committed: Sun Jan 3 22:58:15 2016 -0800
>>
>> ----------------------------------------------------------------------
>>  flexunit-tests.xml                                        |  4 +++-
>>  .../spark/tests/spark/skins/spark/FLEX_34625_Tests.as     | 10 +++++++---
>>  2 files changed, 10 insertions(+), 4 deletions(-)
>> ----------------------------------------------------------------------
>>
>>
>> http://git-wip-us.apache.org/repos/asf/flex-sdk/blob/db54a3f3/flexunit-tests.xml
>> ----------------------------------------------------------------------
>> diff --git a/flexunit-tests.xml b/flexunit-tests.xml
>> index 6475956..dce8cc7 100644
>> --- a/flexunit-tests.xml
>> +++ b/flexunit-tests.xml
>> @@ -77,6 +77,7 @@
>>              toDir="${tests.bin}/report"
>>              haltonfailure="false"
>>              verbose="true"
>> +            debug="true"
>>              localTrusted="true">
>>
>>        <source dir="${project.root}/src"/>
>> @@ -85,7 +86,7 @@
>>          <include name="**/*Tests.as"/>
>>        </testSource>
>>
>> -      <library dir="${FLEXUNIT_HOME}/FlexUnit4/target"/>
>> +      <library 
>> file="${FLEXUNIT_HOME}/FlexUnit4/target/flexunit-4.3.0-20140410-flex_4.12.0.swc"/>
>>        <library dir="${FLEXUNIT_HOME}/FlexUnit4CIListener/target"/>
>>      </flexunit>
>>    </target>
>> @@ -96,6 +97,7 @@
>>              toDir="${tests.bin}/report"
>>              haltonfailure="false"
>>              verbose="true"
>> +            debug="true"
>>              localTrusted="true">
>>
>>        <source dir="${project.root}/src"/>
>>
>> http://git-wip-us.apache.org/repos/asf/flex-sdk/blob/db54a3f3/frameworks/projects/spark/tests/spark/skins/spark/FLEX_34625_Tests.as
>> ----------------------------------------------------------------------
>> diff --git 
>> a/frameworks/projects/spark/tests/spark/skins/spark/FLEX_34625_Tests.as 
>> b/frameworks/projects/spark/tests/spark/skins/spark/FLEX_34625_Tests.as
>> index c040b38..fb0cbac 100644
>> --- a/frameworks/projects/spark/tests/spark/skins/spark/FLEX_34625_Tests.as
>> +++ b/frameworks/projects/spark/tests/spark/skins/spark/FLEX_34625_Tests.as
>> @@ -40,7 +40,11 @@ package spark.skins.spark {
>>          [Before]
>>          public function setUp():void
>>          {
>> -            var _focusManager:FocusManager = new 
>> FocusManager(UIImpersonator.testDisplay as IFocusManagerContainer);
>> +            var _focusManager:FocusManager;
>> +            if (UIImpersonator.testDisplay is IFocusManagerContainer)
>> +                _focusManager = new FocusManager(UIImpersonator.testDisplay 
>> as IFocusManagerContainer);
>> +            else
>> +                _focusManager = 
>> UIImpersonator.testDisplay.parent["document"].focusManager;
>>              _focusManager.showFocusIndicator = true;
>>
>>              _textInput = new TextInput();
>> @@ -59,7 +63,7 @@ package spark.skins.spark {
>>          public function test_focus_skin_with_zero_focus_thickness():void
>>          {
>>              //given
>> -            UIImpersonator.addChild(_textInput);
>> +            UIImpersonator.addElement(_textInput);
>>
>>              //when
>>              _textInput.setStyle("focusThickness", 0);
>> @@ -75,7 +79,7 @@ package spark.skins.spark {
>>          public function test_focus_skin_with_NaN_focus_thickness():void
>>          {
>>              //given
>> -            UIImpersonator.addChild(_textInput);
>> +            UIImpersonator.addElement(_textInput);
>>
>>              //when
>>              _textInput.setStyle("focusThickness", NaN);
>>

Reply via email to