MX integration tests are still failing with a similar message. Note that his
time it’s SetupListAndDataGridTest (not button test which passes).
Edward, do you think you can take a look?
I got the below message in my flash log
C:\Users\yisha\AppData\Roaming\Macromedia\Flash Player\Logs\flashlog.txt
After running
ant mxtests from royale-asjs
RESULT:
scriptName=halo.scripts::ListAndDataGridTestScript&id=SetupListAndDataGridTest&result=fail&elapsed=11&phase=body&started=1628843604291&extraInfo=&msg=ArgumentError:
Error #2024: An object cannot be added as a child of itself.^ at
flash.display::DisplayObjectContainer/addChild()^ at
org.apache.royale.core::UIBase/$sprite_addChild()[/org/apache/royale/0.9.8/org/apache/royale/core/UIBase.as:1584]^
at
org.apache.royale.core::UIBase/addElement()[/org/apache/royale/0.9.8/org/apache/royale/core/UIBase.as:1175]^
at
mx.controls.listClasses::ListBase/addElement()[/org/apache/royale/0.9.8/mx/controls/listClasses/ListBase.as:1032]^
at
org.apache.royale.html.beads::ContainerView/addViewport()[/org/apache/royale/0.9.8/org/apache/royale/html/beads/ContainerView.as:169]^
at org.apache.royale.html.beads::ContainerView/set
strand()[/org/apache/royale/0.9.8/org/apache/royale/html/beads/ContainerView.as:155]^
at org.apache.royale.html.beads::DataContainerView/set
strand()[/org/apache/royale/0.9.8/org/apache/royale/html/beads/DataContainerView.as:75]^
at
org.apache.royale.core::HTMLElementWrapper/addBead()[/org/apache/royale/0.9.8/org/apache/royale/core/HTMLElementWrapper.as:66]^
at
org.apache.royale.core::UIBase/addBead()[/org/apache/royale/0.9.8/org/apache/royale/core/UIBase.as:1148]^
at
global/org.apache.royale.utils::loadBeadFromValuesManager()[/org/apache/royale/0.9.8/org/apache/royale/utils/loadBeadFromValuesManager.as:55]^
at
org.apache.royale.core::UIBase/loadBeads()[/org/apache/royale/0.9.8/org/apache/royale/core/UIBase.as:1419]^
at
org.apache.royale.core::UIBase/addedToParent()[/org/apache/royale/0.9.8/org/apache/royale/core/UIBase.as:1402]^
at
mx.core::UIComponent/addedToParent()[/org/apache/royale/0.9.8/mx/core/UIComponent.as:2365]^
at
mx.controls.listClasses::ListBase/addedToParent()[/org/apache/royale/0.9.8/mx/controls/listClasses/ListBase.as:883]^
at
org.apache.royale.core::UIBase/addElement()[/org/apache/royale/0.9.8/org/apache/royale/core/UIBase.as:1178]^
at
mx.core::Container/addElement()[/org/apache/royale/0.9.8/mx/core/Container.as:1386]^
at
org.apache.royale.utils::MXMLDataInterpreter$/initializeStrandBasedObject()[/org/apache/royale/0.9.8/org/apache/royale/utils/MXMLDataInterpreter.as:307]^
at
org.apache.royale.utils::MXMLDataInterpreter$/generateMXMLArray()[/org/apache/royale/0.9.8/org/apache/royale/utils/MXMLDataInterpreter.as:157]^
at
org.apache.royale.utils::MXMLDataInterpreter$/generateMXMLInstances()[/org/apache/royale/0.9.8/org/apache/royale/utils/MXMLDataInterpreter.as:337]^
at
mx.core::UIComponent/createChildren()[/org/apache/royale/0.9.8/mx/core/UIComponent.as:4166]^
at
mx.core::Container/createChildren()[/org/apache/royale/0.9.8/mx/core/Container.as:1101]^
at
mx.core::UIComponent/initialize()[/org/apache/royale/0.9.8/mx/core/UIComponent.as:4112]^
at
mx.core::UIComponent/addedToParent()[/org/apache/royale/0.9.8/mx/core/UIComponent.as:2382]^
at
mx.core::Container/addedToParent()[/org/apache/royale/0.9.8/mx/core/Container.as:1089]^
at
mx.containers::Box/addedToParent()[/org/apache/royale/0.9.8/mx/containers/Box.as:229]^
at
org.apache.royale.core::UIBase/addElementAt()[/org/apache/royale/0.9.8/org/apache/royale/core/UIBase.as:1209]^
at
mx.core::Container/addElementAt()[/org/apache/royale/0.9.8/mx/core/Container.as:1398]^
at
org.apache.royale.core::StatesWithTransitionsImpl/apply()[/org/apache/royale/0.9.8/org/apache/royale/core/StatesWithTransitionsImpl.as:361]^
at
org.apache.royale.core::StatesWithTransitionsImpl/stateChangeHandler()[/org/apache/royale/0.9.8/org/apache/royale/core/StatesWithTransitionsImpl.as:162]^
at flash.events::EventDispatcher/dispatchEventFunction()^ at
flash.events::EventDispatcher/dispatchEvent()^ at
org.apache.royale.core::UIBase/dispatchEvent()[/org/apache/royale/0.9.8/org/apache/royale/core/UIBase.as:1577]^
at mx.core::UIComponent/set
currentState()[/org/apache/royale/0.9.8/mx/core/UIComponent.as:3550]^
at
SetProperty/doStep()[C:/dev/full_royale_sdk/royale-asjs/mustella/src/main/royale/SetProperty.as:90]^
at
TestStep/execute()[C:/dev/full_royale_sdk/royale-asjs/mustella/src/main/royale/TestStep.as:75]^
at
SetProperty/execute()[C:/dev/full_royale_sdk/royale-asjs/mustella/src/main/royale/SetProperty.as:52]^
at
TestCase/runSteps()[C:/dev/full_royale_sdk/royale-asjs/mustella/src/main/royale/TestCase.as:443]^
at
TestCase/runBody()[C:/dev/full_royale_sdk/royale-asjs/mustella/src/main/royale/TestCase.as:318]^
at
TestCase/runSetup()[C:/dev/full_royale_sdk/royale-asjs/mustella/src/main/royale/TestCase.as:296]^
at
TestCase/runTest()[C:/dev/full_royale_sdk/royale-asjs/mustella/src/main/royale/TestCase.as:264]^
at
UnitTester/runTests()[C:/dev/full_royale_sdk/royale-asjs/mustella/src/main/royale/UnitTester.as:2113]^
at
UnitTester/startTests()[C:/dev/full_royale_sdk/royale-asjs/mustella/src/main/royale/UnitTester.as:2046]^
at
ScriptRunner/runScripts()[C:/dev/full_royale_sdk/royale-asjs/mustella/src/main/royale/ScriptRunner.as:41]^
at
UnitTester$/enterFrameHandler()[C:/dev/full_royale_sdk/royale-asjs/mustella/src/main/royale/UnitTester.as:1253]
From: Edward Stangler<mailto:[email protected]>
Sent: Wednesday, July 21, 2021 7:31 AM
To: Yishay Weiss<mailto:[email protected]>;
[email protected]<mailto:[email protected]>
Subject: Re: Mustella Tests
PR #1134 fixes the issue.
On 7/9/2021 7:37 AM, Edward Stangler wrote:
Yes, I've reproduced with a stand-alone test case based on the tests.
I know what the problem is, now.
The fix is easy, and I've tested it, but I need a little more time with it.
On 7/9/2021 6:53 AM, Yishay Weiss wrote:
Good to know you are working on it. Were you able to run the tests? If not, it
may be enough to run an app with an includeIn and a state change. As you can
see up thread contanerviiew.addviewport is what's failing.
My CLT-L09. On O2.
-------- Original message --------
From: Edward Stangler
Date: Fri, 9 Jul 2021, 05:07
To: Yishay Weiss <[email protected]><mailto:[email protected]>,
[email protected]<mailto:[email protected]>
Subject: Re: Mustella Tests
I think the last few comments in PR 1041 are the same issue. I started to
explain why I made this commit.
Basically, the commit was needed so that I could do more important work in
other areas, to fix several difficult issues. (I have uncommitted work from
that time period.) For one thing, it made debugging much much easier. And it
helped in some other ways. (I understand what Alex mentioned, but I think the
commit was an improvement. But I need to see if I left something out of the
commit. Obviously, if this test error remains, the commit needs to be rolled
back.)
Sorry for the delay in addressing this. I'm working on it right now (trying to
get the tests running on my PC).
On 7/8/2021 7:52 AM, Yishay Weiss wrote:
Looks like reverting 6d1c020577cec69b6207a5b46356b6a2b74f308c fixes it
@Edward Stangler<mailto:[email protected]> , do you think you can have a
look?
Here’s what I needed to do to run Mustella tests:
a. Make sure the following env vars are defined (I’m giving my values for
reference)
AIR_HOME=C:\MoonshineSDKs\Flex_SDK\Flex_4.16.1_AIR_32.0
FLASHPLAYER_DEBUGGER=C:\dev\content_debugger\flashplayer_32_sa_debug.exe
GOOG_HOME=C:\dev\full_royale_sdk\royale-asjs\js\lib\google\closure-library
PLAYERGLOBAL_HOME=C:\dev\player
b. Make sure %PLAYERGLOBAL_HOME%/11.1/playerglobal.swc exists
c. Make sure ~/mm.cfg exists and contains the following line
TraceOutputFileEnable=1
d. cd …/royale-asjs/
e. ant basictests
From: Yishay Weiss<mailto:[email protected]>
Sent: Thursday, July 8, 2021 9:23 AM
To: [email protected]<mailto:[email protected]>
Subject: RE: Mustella Tests
Ok, got it. I do not have a Mac. I’ll look at the Git log for changes to
containerview.
Enjoy your vacation!
From: Alex Harui<mailto:[email protected]>
Sent: Wednesday, July 7, 2021 8:37 PM
To: [email protected]<mailto:[email protected]>
Subject: Re: Mustella Tests
I'm on a family trip and took my windows laptop instead of my Mac. I debugged
into it and now wonder how it passed on the Mac. There were changes to basic
containerview that don't seem right for swf.
Maybe you can debug on Mac in containerview addviewport or maybe Mac isn't
output log and we are looking at old log.
Sent via the Samsung Galaxy A51 5G, an AT&T 5G smartphone
Get Outlook for Android<https://aka.ms/AAb9ysg>
________________________________
From: Yishay Weiss <[email protected]><mailto:[email protected]>
Sent: Tuesday, July 6, 2021 8:56:42 AM
To: [email protected]<mailto:[email protected]>
<[email protected]><mailto:[email protected]>
Subject: RE: Mustella Tests
Did you rule out fp versions and different env vars?
From: Alex Harui<mailto:[email protected]>
Sent: Tuesday, July 6, 2021 9:12 AM
To:
[email protected]<mailto:[email protected]><mailto:[email protected]><mailto:[email protected]>
Subject: Re: Mustella Tests
I got the same error on my Windows machine, but it passed on my Mac. Very
strange because I don't think we have much mac vs win code. Not sure if I'll
have time to debug into it.
-Alex
On 7/5/21, 2:42 AM, "Yishay Weiss"
<[email protected]><mailto:[email protected]> wrote:
Hi Guys,
We’re close to having a release out. The hurdle right now is that Mustella
tests are failing on the CI. Does anyone have time to test if they are passing
locally on their pc? Right now it fails for me with
>scriptName=halo.scripts::ButtonTestScript&id=SetupButtonTest&result=fail&elapsed=13&phase=body&started=1625149537338&extraInfo=&msg=ArgumentError:
Error #2024: An object cannot be added as a child of itself.^
Looking at ButtonTest I think this is caused by a state change that
manifests in an includeIn command. I don't know if includeIn stopped working in
swf (this test passed previously) or if I’m missing some env var on CI.
I will probably revisit this by the end of the week if nobody else beats me
to it.
Thanks.
