Code review comment for lp:~allenap/launchpad/initseries-for-one-night-only-bug-793620

Revision history for this message
Jonathan Lange (jml) wrote :

On Thu, Jun 9, 2011 at 3:43 PM, Gavin Panella
<email address hidden> wrote:
> I hadn't seen AfterPreprocessing before, neat :)
>
> However, AfterPreprocessing only processes the matchee, not the
> matcher, so that lambda doesn't ignore whitespace in the same way as
> EqualsIgnoringWhitespace does. In that respect I think
> EqualsIgnoringWhitespace is less surprising.
>

Ahh, OK. I guess testtools could use a combinator like
AfterPreprocessing that maps both the matcher and the matchee.
Shouldn't have to write a class to do this.

jml

« Back to merge proposal