[infusion] FLUID-4453: Hackish strategy for resolving cross-iframe DOM manipulation issue on IE6. (#167)
Obara, Justin
jobara at ocad.ca
Wed Sep 21 13:05:35 UTC 2011
Thanks for looking into these Antranig. I ran the test 50+ times and did get a single failure on my machine. I think the 200ms timeout is working better.
Thanks
Justin
On 2011-09-21, at 1:10 AM, Antranig Basman wrote:
> Thanks for checking this out, Justin - it really is a deeply unsatisfactory way of testing the component but unfortunately it is hard to do much better without refactoring the base component significantly and/or improving the core framework which we don't have time for right now. I increased the timeout to 200ms in the meantime in my commit, let me know whether it is better enough on that configuration.
> I also pushed the linting fix you mentioned,
> Antranig
>
> On 20/09/2011 11:56, Justin Obara wrote:
>>> + function testToCEnhancement(innerThat, uiOptionsLoader, innerUIOptions) {
>>> + uiOptions = innerUIOptions;
>>> + }
>>> +
>>> + function testToCEnhancement2() {
>>> + fluid.tests.uiOptions.applierRequestChanges(uiOptions, testSettings);
>>> + jqUnit.expect(1);
>>> + // TODO: Very unsatisfactory - the TOC resources are the final thing we wait on, and the
>>> + // event for this is very deeply buried
>>> + setTimeout(function () {
>>> + var container = uiOptions.preview.enhancerContainer;
>>> + var links = $(".flc-toc-tocContainer a", container);
>>> + jqUnit.assertTrue("ToC links created", links.length> 0);
>>> + start();
>>> + }, 100);
>>> + }
>>
>> I ran this test in FF 6 on Mac OS 10.7 and it fails from time to time. Guessing it's a timing issue. Maybe the delay needs to be longer or a different approach taken. It does seem to work most of the time though.
>>
>
More information about the fluid-work
mailing list