Create fail_if_returned_late, behavior is similar to fail_if_returned_early

[oaxlin]

Please allow for stricter checking of test plans. An option already exists for early returns. However running too many can be just as bad.

This pull request gives Test::Class the same control over too many tests that it already had for too few.

[karenetheridge]

note to self: should be in a -TRIAL release first.

[jgoodman]

Hi,

Can we get this patch merged in? I would like to have this feature in my tests.

@karenetheridge: You mentioned "should be in a -TRIAL release first". I am not seeing this as a branch, is this a separate repo? If so, should future pull requests go to that and where would it be located?

Thanks!

[pali]

Hi! I would like to use this fail_if_returned_late feature in my Test::Class::WithStrictPlan module.

@karenetheridge could you review this pull request?

[choroba]

What blocks this from merging, please?

[karenetheridge]

I don't use Test::Class anymore (I never did all that much actually; I don't understand its internals), so I am unable to review and test any proposed patches. I am not sure if Adrian is around anymore either.

I can ship proposed patches as trial releases, but am unable to guarantee any level of quality for them.

Test::Class::Moose is a well-maintained variant though; it may have resolved this issue already.

[adrianh]

Yup - I'm not active in the perl world anymore. Back in the day I tried repeatedly to get this adopted but nobody stood up. If anybody does want to formally adopt the module please do (and remind me of what hoops I need to jump through to make it happen :-)

[jgoodman]

Hey, I could adopt this!

[adrianh]

Sweet! Thanks Kaycie. I'll give you repo access once I remember how to. Chuck me your PAUSE id and I'll add you in as primary maintainer.

[jgoodman]

@adrianh my PAUSE id is jgoodman :)

[adrianh]

I apparently have the memory of an idiot. And Karen is the current owner of the module. @karenetheridge if you don't wanna maintain any more do you wanna transfer to Kaycie ?

[pali]

Any update on this? Btw, @adrianh you are still co-maintainer of Test::Class, see:
https://metacpan.org/permission/distribution/Test-Class

[adrianh]

I'm might be co-maintainer — but I shouldn't be. No longer competent enough to release.

[pali]

@karenetheridge Can you process that maintenance transfer?

[karenetheridge]

No I cannot. I'm not a PAUSE admin.

[pali]

@exodist Hi! According to CPAN you are owner of Test-Class distribution:
https://metacpan.org/permission/distribution/Test-Class

Could you transfer ownership to @jgoodman? See whole issue, neither @adrianh nor @karenetheridge can do it and Test-Class distribution is already unmaintained.

[exodist]

Gladly... and done. Have fun @jgoodman.

[exodist]

Hmm, the transfer did not bump me to a co-maint, it removed me completely. I do not expect to need to do anything, but it might be a good idea to keep me as a co-maint, so if someone wants to add me back on as a co-maint that might be a good idea.

[pali]

Great! @jgoodman can you now process pending pull requests?

[jgoodman]

@exodist I added you back in as a co-maint

@pali So far I am planning on either this Saturday or Sunday to devote time to this module.

[choroba]

The trial release was moved to backpan. Were there any problems?