Stories
Slash Boxes
Comments

NOTE: use Perl; is on undef hiatus. You can read content, but you can't post it. More info will be forthcoming forthcomingly.

use Perl

All the Perl that's Practical to Extract and Report

Test::Class script More | Login | Reply

The Fine Print: The following comments are owned by whoever posted them. We are not responsible for them in any way.

Without JavaScript enabled, you might want to use the classic discussion system instead. If you login, you can remember this preference.

Room for improvement (Score:1)

by Aristotle (5147)

First of all: unless ... else ...? What were you thinking? :)

Secondly: sticking the TEST_DIR removal in file_to_module seems like a violation of separatation of concerns. Yeah, I know this script is small, but it still bugged me [perlmonks.org] when I did it in a small script of my own [perlmonks.org]. My suggestion would be to fix it the same way as I did:
use File::Spec::Functions qw( abs2rel ); # ... # file_to_module which does not concern itself with TEST_DIR # ... ( file_to_module abs2rel $file, TEST_DIR )->require or die $@;

Get More Comments

Stories, comments, journals, and other submissions on use Perl; are Copyright 1998-2006, their respective owners.