Switch 'analyzer' to 'package:test' and test_reflective_loader ^0.1.0.

Switch 'analyzer' to 'package:test' and test_reflective_loader ^0.1.0.

R=brianwilkerson@google.com, paulberry@google.com
BUG=

Committed: https://github.com/dart-lang/sdk/commit/175dad4f1562bdc6c14a99638c446095fc1ab7f1

[kevmoo, 2016-10-06 02:39:34]

kevmoo@google.com changed reviewers:
+ kevmoo@google.com

[kevmoo, 2016-10-06 02:39:35]

DBC:

> pub run test
00:49 +11773: All tests passed!

That felt AWESOME! :-D

[Paul Berry, 2016-10-06 03:01:03]

In order to avoid breaking the bots, you'll also need to update the toplevel
DEPS file like so:

-  "test_reflective_loader_tag": "@0.0.4",
+  "test_reflective_loader_tag": "@0.1.0",

Assuming that change is made, lgtm.

https://codereview.chromium.org/2391423003/diff/1/pkg/analyzer/test/utils.dart
File pkg/analyzer/test/utils.dart (right):

https://codereview.chromium.org/2391423003/diff/1/pkg/analyzer/test/utils.dar...
pkg/analyzer/test/utils.dart:14: void initializeTestEnvironment([path.Context
context]) {}
Since this function doesn't do anything anymore, can we remove it (and all calls
to it)?  It looks like this CL removes some of the calls but not others.

[Brian Wilkerson, 2016-10-06 03:33:54]

Should all of the invocations of defineReflectiveSuite define a name?

Other than that, lgtm.

[scheglov, 2016-10-06 15:46:40]

Description was changed from

==========
Switch 'analyzer' to 'package:test' and test_reflective_loader ^0.1.0.

R=brianwilkerson@google.com, paulberry@google.com
BUG=
==========

to

==========
Switch 'analyzer' to 'package:test' and test_reflective_loader ^0.1.0.

R=brianwilkerson@google.com, paulberry@google.com
BUG=

Committed:
https://github.com/dart-lang/sdk/commit/175dad4f1562bdc6c14a99638c446095fc1ab7f1
==========

[scheglov, 2016-10-06 15:46:42]

Committed patchset #1 (id:1) manually as
175dad4f1562bdc6c14a99638c446095fc1ab7f1 (presubmit successful).