Remove "Remove duplication of test setup" from the todo list.
authorNicholas Clark <nick@ccl4.org>
Fri, 30 Mar 2012 13:54:00 +0000 (15:54 +0200)
committerNicholas Clark <nick@ccl4.org>
Tue, 24 Apr 2012 08:51:55 +0000 (10:51 +0200)
commit5ca9d8f086c5569a14a792ee55c75478496f2a8f
tree913afaf6e3287dcb528a856894e046cbb18ab591
parent78b489b0bf183e0178ccbd0309917683110d8aa3
Remove "Remove duplication of test setup" from the todo list.

It's no longer obvious which files in t/ contain significant amounts of
setup code which is duplicated. Common cases of skip logic have already been
moved into t/test.pl. The suggestion of replacing $Is_Foo checks with %Is is
not a clear-cut win - each $Is_Foo eliminated is only a 1 line saving, at the
cost of another testing idiom the future maintenance programmer has to become
familiar with.

The related suggestion "Migrate t/ from custom TAP generation" remains
something to do. There are still various files in t/ which could use be
converted to use the comparison functions of t/test.pl reducing code size,
improving failure diagnostics and where this eliminates manual test counters,
increasing flexibility for adding tests in the logically correct location
within the file.
Porting/todo.pod