[infusion] FLUID-4453: Hackish strategy for resolving cross-iframe DOM manipulation issue on IE6. (#167)

Antranig Basman antranig.basman at colorado.edu
Wed Sep 21 05:10:01 UTC 2011


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