Fix a couple of minor issues and try to resolve #2740.

Fix a couple of minor issues and try to resolve #2740.

Committed: https://code.google.com/p/dart/source/detail?r=7087

[Bob Nystrom, 2012-04-26 22:02:05]

I'm trying to address http://code.google.com/p/dart/issues/detail?id=2740 here.
The actual fix for the issue should have been in
https://chromiumcodereview.appspot.com/10191027/.

Mads, how were you seeing this issue? Was it from running tests locally? Is it
possible you hadn't synced my change when you hit this?

The actual issue as far as I know is that I wasn't sorting the results of
listing a directory, but the test required sorted output. On my Mac, it always
came back sorted, so it worked. On a couple of others' machines (Linux?) that
didn't seem to be the case.

Regardless, in looking into this, I found a couple of other minor issues that
are cleaned up here in addition to re-enabling pub on the bots.

Sorry for the trouble.

[nweiz, 2012-04-26 23:42:47]

lgtm

https://chromiumcodereview.appspot.com/10230016/diff/1/utils/tests/pub/test_p...
File utils/tests/pub/test_pub.dart (right):

https://chromiumcodereview.appspot.com/10230016/diff/1/utils/tests/pub/test_p...
utils/tests/pub/test_pub.dart:92: return createTempDir('pub-test-sandbox-');
Why?

[Mads Ager (google), 2012-04-27 06:10:24]

LGTM, thanks!

Yes, I was seeing this on my local Linux machine. The good news is that this
issue made us realize that not all tests were running on the VM buildbots. They
now are so we will catch these issues when the changes go in. :-)

[Bob Nystrom, 2012-04-27 15:46:53]

On 2012/04/26 23:42:47, nweiz wrote:
> lgtm
> 
>
https://chromiumcodereview.appspot.com/10230016/diff/1/utils/tests/pub/test_p...
> File utils/tests/pub/test_pub.dart (right):
> 
>
https://chromiumcodereview.appspot.com/10230016/diff/1/utils/tests/pub/test_p...
> utils/tests/pub/test_pub.dart:92: return createTempDir('pub-test-sandbox-');
> Why?

If I run test.py with -m all I see some strange errors if I don't do this. I
believe test.dart may be running different test configurations in parallel which
does horrendously bad things if they happen to be accessing the same files. This
ensures each invocation is nicely sandboxed.