Test-More / TB2

Test::Builder version 2, the next generation of building testing modules in Perl
Other
1 stars 0 forks source link

Remove new TestState hack from Test::Builder->current_test #165

Closed schwernbot closed 10 years ago

schwernbot commented 10 years ago

From: @schwern Date: Friday Nov 04, 2011 at 02:58 GMT Orig: https://github.com/Test-More/test-more/issues/171

Test::Builder->current_test has a hack where it creates a new event coordinator just so it can pass events to the history without other event watchers seeing them.

Use $history->receive_event instead.

schwernbot commented 10 years ago

From: @jkeenan Date: Sunday Nov 06, 2011 at 02:14 GMT Orig: https://github.com/Test-More/test-more/issues/171#issuecomment-2643520

Is 'receive_event' an existing method? If so, where is it defined?

schwernbot commented 10 years ago

From: @jkeenan Date: Sunday Nov 06, 2011 at 02:16 GMT Orig: https://github.com/Test-More/test-more/issues/171#issuecomment-2643530

Oh, it appears that 'receive_event' has already been superseded by 'accept_event', as per https://github.com/schwern/test-more/issues/165.

schwernbot commented 10 years ago

From: @schwern Date: Sunday Nov 06, 2011 at 07:46 GMT Orig: https://github.com/Test-More/test-more/issues/171#issuecomment-2644312

receive_event is defined in EventWatcher per #164. It's quite recent, added on Wednesday.

165 has not been done, so receive_event is still what you want.