tree ea9b940401e193bc7aec76c5f95b85c8e1126899
parent 3f49c6d4c3b028ffb39d83fb42574ea95232a0fc
author Greg Edelston <gredelston@google.com> 1580321432 -0700
committer Commit Bot <commit-bot@chromium.org> 1580752404 +0000

E2E summarizer: Put test-reading logic in function

Previously, the only purpose of the coverage summarizer was to collect
information about existing Autotest tests. Therefore, it made sense to
keep the logic about reading tests in main().

However, as we look forward to expanding this script to also account for
unwritten tests, this is no longer the only purpose of main().
This CL improves readability by putting the test-reading logic in a
separate function.

TEST=go run fw_e2e_coverage_summarizer.go
BUG=b:147815722

Change-Id: Ib0d94aaf9aaae90d608baadfd6bf72c85afd5aa2
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crostestutils/+/2027903
Tested-by: Greg Edelston <gredelston@google.com>
Reviewed-by: Kevin Shelton <kmshelton@chromium.org>
Commit-Queue: Greg Edelston <gredelston@google.com>