Fix a couple test timeouts.

Fix a couple test timeouts.

R=rnystrom@google.com

Committed: https://github.com/dart-lang/json_rpc_2/commit/809dba9bc9e95de0062cc2420319f1a745ef5805

[Bob Nystrom, 2015-01-21 23:14:30]

https://codereview.chromium.org/864983002/diff/1/test/server/utils.dart
File test/server/utils.dart (right):

https://codereview.chromium.org/864983002/diff/1/test/server/utils.dart#newco...
test/server/utils.dart:66: // Should not be called.
I'm having trouble following this. How can this fail?

[nweiz, 2015-01-22 00:05:45]

https://codereview.chromium.org/864983002/diff/1/test/server/utils.dart
File test/server/utils.dart (right):

https://codereview.chromium.org/864983002/diff/1/test/server/utils.dart#newco...
test/server/utils.dart:66: // Should not be called.
On 2015/01/21 23:14:30, Bob Nystrom wrote:
> I'm having trouble following this. How can this fail?

It fails if the future completes, since [expectAsync] specifies that it expects
the callback to be called zero times.

[Bob Nystrom, 2015-01-22 00:08:22]

lgtm

https://codereview.chromium.org/864983002/diff/1/test/server/utils.dart
File test/server/utils.dart (right):

https://codereview.chromium.org/864983002/diff/1/test/server/utils.dart#newco...
test/server/utils.dart:66: // Should not be called.
On 2015/01/22 00:05:45, nweiz wrote:
> On 2015/01/21 23:14:30, Bob Nystrom wrote:
> > I'm having trouble following this. How can this fail?
> 
> It fails if the future completes, since [expectAsync] specifies that it
expects
> the callback to be called zero times.

Ah, can you add a comment to that effect?

[nweiz, 2015-01-22 00:15:24]

https://codereview.chromium.org/864983002/diff/1/test/server/utils.dart
File test/server/utils.dart (right):

https://codereview.chromium.org/864983002/diff/1/test/server/utils.dart#newco...
test/server/utils.dart:66: // Should not be called.
On 2015/01/22 00:08:22, Bob Nystrom wrote:
> On 2015/01/22 00:05:45, nweiz wrote:
> > On 2015/01/21 23:14:30, Bob Nystrom wrote:
> > > I'm having trouble following this. How can this fail?
> > 
> > It fails if the future completes, since [expectAsync] specifies that it
> expects
> > the callback to be called zero times.
> 
> Ah, can you add a comment to that effect?

Done.

[nweiz, 2015-01-22 00:16:24]

Committed patchset #2 (id:20001) manually as
809dba9bc9e95de0062cc2420319f1a745ef5805 (presubmit successful).