Alternative Error Handling

[aljcalandra]

• Added an error class
• Made validation logic clearer in one case

[coveralls]

coverage: 100.0%. remained the same when pulling 31745ea on aljcalandra:alternative_error_handling into 756730a on Accruent:master.

[aljcalandra]

Is there anything that can be done for the failing tests or is this issue caused elsewhere?

[Wolfe1]

1. Thanks for the pull request long time no see ♥
2. That Edge test is just flakiness, not sure about the "generate_report" job either just thats just broken it looks like.
3. The chrome tests are actual "failures" caused by the code change (They passed today for a dependabot update from Robotframework). Since the error messages changed in your pull request, the expected values for those tests need to be modified to match.

[aljcalandra]

Oof just saw the response, I'll go through and fix that up when I get a chance

Oh also, fun stuff, w3schools has different css for edge vs chrome.

[aljcalandra]

The token used for the generate_report job does not have write permissions for issues/pullrequests so I think it'll fail until that gets updated.
Alternatively it might be worth looking into the replacing it with the summary action.

[Wolfe1]

Appreciate the effort, thanks!