The 'errors' record for series now counts all errors for tests/series.

[Paul-Ferrell]

I've noticed that the 'errors' number when running pav series status doesn't include all test errors, like timeouts and other errors. It should with this change.

Code review checklist:

• Code is generally sensical and well commented
• Variable/function names all telegraph their purpose and contents
• Functions/classes have useful doc strings
• Function arguments are typed
• Added/modified unit tests to cover changes.
• New features have documentation added to the docs.
• New features and backwards compatibility breaks are noted in the RELEASE.md

[hwikle-lanl]

Using str.__contains__ here feels a bit kludgey and error-prone. A better solution would be to implement a mechanism to separate statuses into ERROR, CANCELLED, and TIMEOUT classes, either by:

a) Creating something like a class field on TestStatusInfo that takes on an enum value and testing against that; or

b) Implementing three static methods on StatesStruct that return sets of ERROR, CANCELLED, and TIMEOUT statuses respectively and then checking whether status.state is contained within each set (or simply the union of the three sets).

[hwikle-lanl]

The second option feels cleaner to me.

[hwikle-lanl]

Ditto here.