Code review comment for lp:~parthm/bzr/300062-better-handling-for-invalid-ignore-pattern

Oh, and one more thing: the tests you're writing are very far away
from the code you're testing; please consider using tests for the
specific code, not blackbox tests. blackbox tests should really only
be needed when you're changing code in builtins.py or the few other
command-class containing files.

-Rob