After the last change your statement is totally right. :)
On 29/05/2014 12:37 AM, "Victor Savkin" <notifications@github.com> wrote:

> In lib/src/common/unittest_backend.dart:
>
> > @@ -160,17 +160,23 @@ _isFalsy(v) => v == null ? true: v is bool ? v == false : false;
> >
> >  /// Matches exceptions against a [Type] and a message
> >  class ExceptionMatcher extends unit.Matcher {
> > -  final String _message;
> > +  final Pattern _messageOrRegExp;
>
> I'd just call it _message.
>
>    - It is called message in other parts of the framework.
>    - IMO pattern does not add any new information.
>    - _messageOrRegExp is kind of confusing. RegExp isn't mentioned
>    anywhere in this matcher. So calling it _messageOrRegExp, in my view,
>    implies that RegExp is handled specially, which it is not.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/vsavkin/guinness/pull/10/files#r13133914>
> .
>