see comments on https://github.com/10gen/mongo/pull/28231 for discussion

We use the testingProctor to decide whether a given finding should be a warning or an error, because we only really care about those findings in our tests but don't foresee end-users caring as much.

Alternatives:

 - always report one way or another

 - move the check to results where we translate to bson