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