Fix some warnings in pub and pkg packages.

pkg/unittest/lib/unittest.dart

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /**
  * A library for writing dart unit tests.
  *
  * To import this library, install the
  * [unittest package](http://pub.dartlang.org/packages/unittest) via the pub
  * package manager. See the [Getting Started](http://pub.dartlang.org/doc)
@@ skipping 187 matching lines @@
  */
 String _currentGroup = '';
 
 /** Separator used between group names and test names. */
 String groupSep = ' ';
 
 /** Tests executed in this suite. */
 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;
 
 /** Teardown function called after each test in a group */
 Function _testTeardown;
 
 int _currentTestCaseIndex = 0;
 
 /** [TestCase] currently being executed. */
@@ skipping 638 matching lines @@
 }
 
 /** Enable a test by ID. */
 void enableTest(int testId) => _setTestEnabledState(testId, true);
 
 /** Disable a test by ID. */
 void disableTest(int testId) => _setTestEnabledState(testId, false);
 
 /** Signature for a test function. */
 typedef dynamic TestFunction();