Bug #50462 for JE: scrollTo property is undefined

Tue Oct 13 15:27:26 2009 MARKSTOS [...] cpan.org - Ticket created

Subject:

scrollTo property is undefined

Thanks for this module. Today I tried to test it against the jquery test suite: http://jquery.com/test I would be curious if you have tried it running it the full jQuery test suite. It ran for a couple minutes and then I stopped it and tried what appeared to be a simpler test suite, which it failed on. Here's what appears to be the bug: To reproduce: my $w = WWW::Scripter->new; $w->use_plugin('Ajax'); # packaged separately $w->get('http://openjsan.org/src/t/th/theory/Test.Simple- 0.29/tests/index.html'); print $w->content; ### The result: "TypeError: The object's 'scrollTo' property (undefined) is not a function at line 122."

Sun Oct 18 20:05:06 2009 $_ = 'spro^^*%*^6ut# [...] &$%*c>#!^!#&!pan.org'; y/a-z.@//cd; print - Correspondence added

On Tue Oct 13 15:27:26 2009, MARKSTOS wrote: Show quoted text

> Thanks for this module. > To reproduce: > > my $w = WWW::Scripter->new; > $w->use_plugin('Ajax'); # packaged separately > $w->get('http://openjsan.org/src/t/th/theory/Test.Simple- > 0.29/tests/index.html'); > > print $w->content; > > ### > > The result: > > "TypeError: The object's 'scrollTo' property (undefined) is not a > function at line 122."

I’ve fixed this one, but Test.Simple uncovered many other bugs. I don’t have time to fix them all today, but you can see what I’ve done so far in the change logs for HTML::DOM 0.033, JE 0.039 and WWW::Scripter 0.008.

Sun Oct 18 20:05:07 2009 The RT System itself - Status changed from 'new' to 'open'

Mon Oct 19 09:48:23 2009 mark [...] summersault.com - Correspondence added

Subject:	Re: [rt.cpan.org #50462] scrollTo property is undefined
Date:	Mon, 19 Oct 2009 09:48:00 -0400
To:	bug-JE [...] rt.cpan.org
From:	Mark Stosberg <mark [...] summersault.com>

On Sun, 18 Oct 2009 20:05:08 -0400 "Father Chrysostomos via RT" <bug-JE@rt.cpan.org> wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=50462 > > > On Tue Oct 13 15:27:26 2009, MARKSTOS wrote:

> > Thanks for this module. > > To reproduce: > > > > my $w = WWW::Scripter->new; > > $w->use_plugin('Ajax'); # packaged separately > > $w->get('http://openjsan.org/src/t/th/theory/Test.Simple- > > 0.29/tests/index.html'); > > > > print $w->content; > > > > ### > > > > The result: > > > > "TypeError: The object's 'scrollTo' property (undefined) is not a > > function at line 122."

> > I’ve fixed this one, but Test.Simple uncovered many other bugs. I don’t have time to fix them > all today, but you can see what I’ve done so far in the change logs for HTML::DOM 0.033, JE > 0.039 and WWW::Scripter 0.008. >

Thanks for your work on this. Mark

Sun Oct 25 17:48:52 2009 $_ = 'spro^^*%*^6ut# [...] &$%*c>#!^!#&!pan.org'; y/a-z.@//cd; print - Correspondence added

Wow! Test.Simple found a lot of bugs. It’s slow, and the whitespace in the text output is not perfect, but it works: perl -MTime::HiRes=sleep -e 'use WWW::Scripter; agent_alias{$w=new WWW::Scripter}"Mac Safari"; $w->use_plugin(Ajax); $w->get("http://localhost/ts/index.html"); for($w){ sleep .1, $_->check_timers while $_->count_timers }; print $w->content(format=>text)' async.js........... ok bad_plan.js........ ok buffer.js.......... ok builder.js......... ok builder_death.js... ok circular_data.js... ok curr_test.js....... ok details.js......... ok diag.js............ ok instance.js........ ok jsan.js............ ok harness.js......... ok more.js............ ok output.js.......... ok simple.html........ ok simpler.html....... ok simple.js.......... ok test_result.js..... ok All tests successful, 6 subtests skipped. Files=18, Tests=294, 56 seconds This is with JE 0.040, WWW::Scripter 0.009, WWW::Scripter::Plugin::JavaScript (wspjs) 0.004 and HTML::DOM 0.034. Test.Simple assumes that any browser that does not match /MSIE|Safari|Opera|Konqueror/ has a Firefox-style document.open. This method is different in every browser. WWW::Scripter matches Safari, because it’s the easiest to implement and the closest to DOM 2. That’s why I set the agent_alias.

Sun Oct 25 17:48:54 2009 $_ = 'spro^^*%*^6ut# [...] &$%*c>#!^!#&!pan.org'; y/a-z.@//cd; print - Status changed from 'open' to 'resolved'

Mon Oct 26 10:01:05 2009 mark [...] summersault.com - Correspondence added

Subject:	Re: [rt.cpan.org #50462] scrollTo property is undefined
Date:	Mon, 26 Oct 2009 10:00:27 -0400
To:	bug-JE [...] rt.cpan.org
From:	Mark Stosberg <mark [...] summersault.com>

On Sun, 25 Oct 2009 17:48:53 -0400 "Father Chrysostomos via RT" <bug-JE@rt.cpan.org> wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=50462 > > > Wow! Test.Simple found a lot of bugs. > > It’s slow, and the whitespace in the text output is not perfect, but it works:

Great! Show quoted text

> Test.Simple assumes that any browser that does not match /MSIE|Safari|Opera|Konqueror/ > has a Firefox-style document.open. This method is different in every browser. WWW::Scripter > matches Safari, because it’s the easiest to implement and the closest to DOM 2. That’s why I > set the agent_alias.

Good to know. It sounds like this is an area where Test.Simple could improved, by checking for the actual functionality instead of checking the user agent strings. I expect you'd be able to formulate the details of a bug report better than I would. The author to e-mail about it would be David Wheeler <david@kineticode.com>. Thanks again for these improvements! Mark

Mon Oct 26 10:01:06 2009 The RT System itself - Status changed from 'resolved' to 'open'

Sun Nov 01 15:07:38 2009 $_ = 'spro^^*%*^6ut# [...] &$%*c>#!^!#&!pan.org'; y/a-z.@//cd; print - Correspondence added

On Mon Oct 26 10:01:05 2009, mark@summersault.com wrote: Show quoted text

> On Sun, 25 Oct 2009 17:48:53 -0400 > "Father Chrysostomos via RT" <bug-JE@rt.cpan.org> wrote: >

> > <URL: https://rt.cpan.org/Ticket/Display.html?id=50462 > > > > > Wow! Test.Simple found a lot of bugs. > > > > It’s slow, and the whitespace in the text output is not perfect, but

> it works: > > Great! >

> > Test.Simple assumes that any browser that does not match

> /MSIE|Safari|Opera|Konqueror/

> > has a Firefox-style document.open. This method is different in every

> browser. WWW::Scripter

> > matches Safari, because it’s the easiest to implement and the

> closest to DOM 2. That’s why I

> > set the agent_alias.

> > Good to know. It sounds like this is an area where Test.Simple could > improved, by checking for the actual functionality instead of checking > the user agent strings. > > I expect you'd be able to formulate the details of a bug report better > than I would. The author to > e-mail about it would be David Wheeler <david@kineticode.com>.

Done. Show quoted text

> > Thanks again for these improvements! > > Mark

Sun Nov 01 15:07:43 2009 $_ = 'spro^^*%*^6ut# [...] &$%*c>#!^!#&!pan.org'; y/a-z.@//cd; print - Status changed from 'open' to 'resolved'