Fix some warnings in pub and pkg packages.

pkg/unittest/lib/unittest.dart

Index: pkg/unittest/lib/unittest.dart
diff --git a/pkg/unittest/lib/unittest.dart b/pkg/unittest/lib/unittest.dart
index efe6f8d40f7535b34e12fe33d791b47535a7ab53..24efe2c3901da0eb24a218b8af6e564b4f8a396a 100644
--- a/pkg/unittest/lib/unittest.dart
+++ b/pkg/unittest/lib/unittest.dart
@@ -205,7 +205,7 @@ String groupSep = ' ';
 final List<TestCase> _testCases = new List<TestCase>();
 
 /** Get the list of tests. */
-final List<TestCase> testCases = new UnmodifiableListView(_testCases);
+final List<TestCase> testCases = _testCases;

[gram, 2013/04/15 23:25:55]

If the unmodifiable stuff (which KevMoo added) is bad, then rather than the
above it would probably be better to just make testCases a getter for
_testCases, which is what it was originally. 

[nweiz, 2013/04/15 23:41:16]

I've just made it a property, since it's final either way. I've also added a
TODO to present an unmodifiable view when that's possible.

 
 /** Setup function called before each test in a group */
 Function _testSetup;