fix: forbidUnknownValues should default true when validatorOptions undefined

[cduff]

Description

forbidUnknownValues is supposed to default to true. This wasn't the case when validatorOptions was undefined. This PR corrects that and adds a test for it. It also corrects some related code doc.

Checklist

• the pull request title describes what this PR does (not a vague title like Update index.md)
• the pull request targets the default branch of the repository (develop)
• the code follows the established code style of the repository
  • npm run prettier:check passes
  • npm run lint:check passes
• tests are added for the changes I made (if any source code was modified)
• documentation added or updated
• I have run the project locally and verified that there are no errors

Fixes

fixes #1906

[braaar]

A few changes, but this looks good!

[cduff]

Thanks for your review @braaar. I just made changes based on your feedback. Hopefully it's good to merge now?

[braaar]

You should maybe add a test where you pass {} as the options object

[cduff]

You should maybe add a test where you pass {} as the options object

@braaar Good idea. Done.

[cduff]

@braaar any other changes needed to get this merged?

[braaar]

@braaar any other changes needed to get this merged?

Probably not, but we'll have to wait for @NoNameProvided's review

[cduff]

@braaar I just made another commit based on your feedback. Let me know if anything else is required to get this merged.

[cduff]

Any chance of getting this small PR reviewed & merged @NoNameProvided?

[cduff]

@braaar @NoNameProvided Any chance of getting this small PR reviewed & merged?

[braaar]

Thanks for the contribution!

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.