Bug/3658 add enzyme test for objectmodel

[feenr]

Description

• Add test to ensure that model properties are listed in the order specified in the schema
• Add test to ensure that collapsible header is generated when rendering ObjectModel
• Update CONTRIBUTING.md to specify the correct node version as a project pre-req

Motivation and Context

• Fixes Add enzyme test for <ObjectModel /> schema.properties rendering #3658

How Has This Been Tested?

• Only test changes. Ensured all tests pass before submitting pull request.

Screenshots (if appropriate):

Types of changes

• No code changes (changes to documentation, CI, metadata, etc)
• Dependency changes (any modification to dependencies in package.json)
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

Not sure if there's a code style document, but I followed the style of an existing test.

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

[shockey]

Merged!

Thanks for the PR, @feenr! Your test case looked great, I wouldn't have thought it was your first contribution if GitHub hadn't told me 😄

[feenr]

Thanks for reviewing and accepting. I had been using the swagger-ui docker image a bit at work and am interested to keep contributing. Is there a way I can claim issues, other than just commenting on them?

[webron]

@feenr Thanks for the offer! GitHub is a bit limited in that sense, so the best approach at the moment would indeed be adding a comment on a ticket.