Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(349)

Issue 12224053: Updated contains and isEmpty matches to use iterables. (Closed)

Created:
7 years, 10 months ago by butlermatt
Modified:
7 years, 10 months ago
Reviewers:
gram
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Updated contains and isEmpty matches to use iterables. Added two tests for iterables. Added missing tests for StateErrors matchers. BUG=http://dartbug.com/8159

Patch Set 1 #

Total comments: 1

Patch Set 2 : Update doc-comments #

Total comments: 2

Patch Set 3 : Fixed typo per comment. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+73 lines, -7 lines) Patch
M pkg/unittest/lib/src/core_matchers.dart View 1 2 3 chunks +9 lines, -7 lines 0 comments Download
M pkg/unittest/test/matchers_test.dart View 3 chunks +64 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
butlermatt
https://codereview.chromium.org/12224053/diff/1/pkg/unittest/lib/src/core_matchers.dart File pkg/unittest/lib/src/core_matchers.dart (right): https://codereview.chromium.org/12224053/diff/1/pkg/unittest/lib/src/core_matchers.dart#newcode594 pkg/unittest/lib/src/core_matchers.dart:594: return item.contains(_expected); I changed this to contains(), to allow ...
7 years, 10 months ago (2013-02-07 15:30:28 UTC) #1
gram
lgtm Thanks very much. Just please fix the comment typo (not yours, but it may ...
7 years, 10 months ago (2013-02-07 17:35:23 UTC) #2
butlermatt
https://codereview.chromium.org/12224053/diff/3001/pkg/unittest/lib/src/core_matchers.dart File pkg/unittest/lib/src/core_matchers.dart (right): https://codereview.chromium.org/12224053/diff/3001/pkg/unittest/lib/src/core_matchers.dart#newcode576 pkg/unittest/lib/src/core_matchers.dart:576: * for [Map]s is means the map has the ...
7 years, 10 months ago (2013-02-07 17:50:15 UTC) #3
gram
On 2013/02/07 17:50:15, butlermatt wrote: > https://codereview.chromium.org/12224053/diff/3001/pkg/unittest/lib/src/core_matchers.dart > File pkg/unittest/lib/src/core_matchers.dart (right): > > https://codereview.chromium.org/12224053/diff/3001/pkg/unittest/lib/src/core_matchers.dart#newcode576 > ...
7 years, 10 months ago (2013-02-07 21:44:08 UTC) #4
butlermatt
On 2013/02/07 21:44:08, gram wrote: > On 2013/02/07 17:50:15, butlermatt wrote: > > > https://codereview.chromium.org/12224053/diff/3001/pkg/unittest/lib/src/core_matchers.dart ...
7 years, 10 months ago (2013-02-07 21:48:04 UTC) #5
butlermatt
7 years, 10 months ago (2013-02-07 21:49:17 UTC) #6
On 2013/02/07 21:48:04, butlermatt wrote:
> On 2013/02/07 21:44:08, gram wrote:
> > On 2013/02/07 17:50:15, butlermatt wrote:
> > >
> >
>
https://codereview.chromium.org/12224053/diff/3001/pkg/unittest/lib/src/core_...
> > > File pkg/unittest/lib/src/core_matchers.dart (right):
> > > 
> > >
> >
>
https://codereview.chromium.org/12224053/diff/3001/pkg/unittest/lib/src/core_...
> > > pkg/unittest/lib/src/core_matchers.dart:576: * for [Map]s is means the map
> has
> > > the key, and for [Iterable]s
> > > On 2013/02/07 17:35:23, gram wrote:
> > > > While you're at it can you correct the comment? "is means" => "it means"
> > > 
> > > Done.
> > 
> > Thanks. Can you commit or do you need me to?
> 

I can not commit, so will need you to please. Thanks, Matt

Powered by Google App Engine
This is Rietveld 408576698