On Jun 29, 2007, at 12:28 PM, Bernhard Schmalhofer via RT wrote:
James Keenan via RT schrieb:
There was code in several of the test files in the reconfigure/
branch which was repeated. At
the hackathon, David Adler refactored it into a subroutine which I
then placed in new file
Auxiliary.pm. That file is contained in this patch along with a
revised t/104-init_miniparrot.t
and appropriate MANIFEST revisions.
Please review. Thank you very much.
kid51
Reducing code redundancy is good idea. My suggestion is to call the
packaga something
like Parrot::Test::Configure and accordingly move the file to
lib/Parrot/Test/Configure.pm.
Okay. Let's hold this patch for the time being. In the reconfigure/
branch we're working our way through 56 configuration steps and 56
corresponding RT tickets. Chances are good that we'll develop some
additional subroutines for shortcuts in testing. We'll see what
develops as the best way to package these up, then submit a more
refined module to trunk.
kid51
