[perl #128856] DirHandle: Improve test coverage and providedescriptions

On Mon Sep 05 05:07:59 2016, tonyc wrote:
> On Tue Aug 30 09:22:45 2016, jkeenan wrote:
> > 
> > Tony C:  Ping re second patch.
> 
> There's some spelling errors in the test comments:
> 
> 
> + "Calling read method after failed open method returns faluse value"); 
>                                                          *****
> 
> repeated in 2 other places.
> 
> As to DESTROY, all you can really do to test it is to check for resource
> leaks, which is more in the domain of tools like valgrind.
> 
> Testing the return value of DESTROY doesn't really check that any resources
> have been released.
> 
> Tony

Please review replacement patch attached.

Thank you very much.
-- 
James E Keenan (jkeenan@cpan.org)

---
via perlbug:  queue: perl5 status: open
https://rt.perl.org/Ticket/Display.html?id=128856

• [perl #128856] DirHandle: Improve test coverage and providedescriptions by James E Keenan via RT
• [perl #128856] DirHandle: Improve test coverage and providedescriptions by James E Keenan via RT
• [perl #128856] DirHandle: Improve test coverage and providedescriptions by Tony Cook via RT