commit b921089cf93ae4728ad17ee075d5f3ede6785a08
author stuartmorgan <stuartmorgan@google.com> Tue Jun 22 13:32:03 2021 -0700
committer GitHub <noreply@github.com> Tue Jun 22 13:32:03 2021 -0700
tree 02ad7fdfd9940ead628dd3fdb35b8c0220aa8959
parent 13e5b5b34c151aaa29cccd14ec578914dd7ac221

[flutter_plugin_tools] Add a new base command for looping over packages (#4067)

Most of our commands are generally of the form:
```
  for (each plugin as defined by the tool flags)
    check some things for success or failure
  print a summary all of the failing things
  exit non-zero if anything failed
```

Currently all that logic not consistent, having been at various points copied and pasted around, modified, in some cases rewritten. There's unnecessary boilerplate in each new command, and there's unnecessary variation that makes it harder both to maintain the tool, and to consume the test output:
- There's no standard format for separating each plugin's run to search within a log
- There's no standard format for the summary at the end
- In some cases commands have been written to ToolExit on failure, which means we don't actually get the rest of the runs

This makes a new base class for commands that follow this structure to use, with shared code for all the common bits. This makes it harder to accidentally write new commands incorrectly, easier to maintain the code, and lets us standardize output so that searching within large logs will be easier.

This ports two commands over as a proof of concept to demonstrate that it works; more will be converted in follow-ups.

Related to https://github.com/flutter/flutter/issues/83413