Bug #55454 for POE-Loop-EV: This module doesn't take advantage of POE::Test::Loops

Wed Mar 10 21:56:21 2010 APOCAL [...] cpan.org - Ticket created

Subject:

This module doesn't take advantage of POE::Test::Loops

Hello, While trying to track down some bugs in POE::Loop::Glib I updated the POE::Test::Loops code. I wanted to make sure the new test would still pass on EV, and I tried to run the tests in this module. To my surprise, the new test didn't show up and I tried digging around a bit. Seems like you inlined the generated tests directly into the distro. I assume you had your reasons in the past, but doing it this way means you don't benefit from new releases of PTL! What you would need to do is to use the "poe-gen-tests" program to generate tests during runtime... I hope this can be resolved soon so we get more test stuff running :) -- ~Apocalypse

Wed Mar 24 02:19:35 2010 APOCAL [...] cpan.org - Correspondence added

On Wed Mar 10 21:56:21 2010, APOCAL wrote: Show quoted text

> Hello, > > While trying to track down some bugs in POE::Loop::Glib I updated the > POE::Test::Loops code. I wanted to make sure the new test would still > pass on EV, and I tried to run the tests in this module. > > To my surprise, the new test didn't show up and I tried digging around > a bit. Seems like you inlined the generated tests directly into the > distro. I assume you had your reasons in the past, but doing it this way > means you don't benefit from new releases of PTL! What you would need to > do is to use the "poe-gen-tests" program to generate tests during

runtime... Show quoted text

> > I hope this can be resolved soon so we get more test stuff running :)

I tried digging around a bit and realized the reason for this. You need to select a different EV backend and run the tests against them all. I would like to improve POE::Test::Loops so we can do that kind of "custom options"! Do you have a script that you used to generate those tests in the first place? That would be helpful as a reference point! Thanks again! -- ~Apocalypse

Wed Mar 24 02:19:36 2010 APOCAL [...] cpan.org - Status changed from 'new' to 'open'

Wed Mar 24 03:10:01 2010 APOCAL [...] cpan.org - Correspondence added

Oh boy, it sure is fun replying to myself eh :) As usual dngor++ had an excellent idea to work around PTL's limitation for EV: <Apocalypse> dngor: Maybe we should update PTL to accomodate this: https://rt.cpan.org/Ticket/Display.html?id=55454 <Apocalypse> But that would require a bit of redesign on how PTL generates the tests <@dngor> Make custom subclasses of EV that prepare differnt sub-loops? Target tests against them? <Apocalypse> Yeah that would be one way to do it, and probably the least invasive... <Apocalypse> PoLo-EV-Tester-Devpoll, PoLo-EV-Tester-Kqueue, etc -- ~Apocalypse

Fri Jan 16 14:12:48 2015 DBOOK [...] cpan.org - Correspondence added

Added POE::Test::Loops testing in 0.10

Fri Jan 16 14:12:49 2015 DBOOK [...] cpan.org - Status changed from 'open' to 'resolved'

Fri Jan 16 14:12:49 2015 DBOOK [...] cpan.org - Fixed in 0.10 added