Refactor sensu_check check_hooks property

[treydock]

Pull Request Checklist

Description

Make sensu_check check_hooks property match what is actually output by sensuctl.

Related Issue

Relates to #901. Upstream documentation bug here: sensu/sensu-docs#678

Motivation and Context

The type sensu_check needs to treat check_hooks property the way it's handled by sensuctl. Previously it was just expecting an array of hooks which isn't correct as hooks associated to a check also have a type like critical or warning. This change ensures valid check_hooks are defined.

[treydock]

I will update acceptance tests to ensure that valid check_hooks are handled correctly.

[treydock]

Acceptance tests updated and passing.

[treydock]

I have local changes that are more complex but valid use of check_hooks and they are failing, this PR is not complete.

[treydock]

Looks like can't use PuppetX::Sensu::ArrayProperty as parent for a property when elements are hashes. I'll have to check if this mistake was made else where. I verified with vagrant that the order of array elements in JSON defining a check is the order that is output when listing checks so this should work nicely without comparing should/is with a sort.

[ghoneycutt]

@treydock could you please rebase so this will merge

[ghoneycutt]

Rebased in #962