Move tools tests into a general.shard directory in preparation to changing how we shard tools tests (#36108)

This allows us to override the behavior internally.

* enable lint unnecessary_new

* fix tests

* fix tests

* fix tests

* Upgrade everything except matcher.
* Roll matcher (and test)
* Adjust tests that depend on flutter:test directly to depend on a shim
* Require use of package:test shim and remove other references to package:test

`adb` can sometimes hang, which will in turn hang the Dart isolate if
we're using `Process.runSync()`. This changes many of the `Device` methods
to return `Future<T>` in order to allow them to use the async process
methods. A future change will add timeouts to the associated calls so
that we can properly alert the user to the hung `adb` process.

This is work towards #7102, #9567

* improve device matching error message
fixes https://github.com/flutter/flutter/issues/5675

* fix getDeviceById to match exact name
fixes https://github.com/flutter/flutter/issues/5508

And fix a zillion omissions this uncovered.

rename to printTrace(), printStatus(), and printError()