Standardize the cached Constraint results to match action results

[Mythicaeda]

• Tickets addressed: Closes Schema Mismatch Between Cached Constraint Run and Action Response #1150
• Review: By commit
• Merge strategy: Merge (no squash)

Description

Updated the parser for constraint violations so that the cached run has the same key names as the action response.

Verification

Wrote a JSON parser while working on #1057 that successfully parses both the checkConstraints action response and the cached values from constraint_run.

Documentation

No docs were invalidated

Future work

None

[JoelCourtney]

@jeffpamer FYI this will impact your UI code

[JoelCourtney]

Don't merge until the UI has a PR to keep up to date

[jeffpamer]

Looks like this should be a pretty small change on our end if I'm interpreting this correctly. I'll try to get a UI PR up this afternoon or first thing tomorrow.

[jeffpamer]

Upon further investigation this does not appear to be a breaking change for the UI as we are currently not reading anything out of constraint_run. If we do start using the cached run results in the future, it looks like after this change the schema will match the shape we are already expecting for constraint check results.