Translatable message when invalid keyword used for collection attribute

[njr-11]

This PR contains a couple of different changes. I started out adding javadoc/comments to some of our classes that were missing them. While doing that, I noticed two minor issues and addressed them in this PR. One is that we have no tests for the Object methods of repositories, despite some of them being provided by an invocation handler. I added tests for them. Another is that I found a partially written english-only error message on an error path that we had forgotten to mark with a TODO comment. I improved the message to include more information and added it to the nlsprops file to be handled the same as our other messages. The Condition enumeration also gets some updates to have the names of enumeration constants exactly match Query by Method Name keywords so we can have less hard coded Strings and hard coded length values in the code.

• I have considered the risk of behavior change or other zero migration impact (https://github.com/OpenLiberty/open-liberty/wiki/Behavior-Changes).
• If this PR fixes an Issue, the description includes "Fixes #FILLMEIN" or "Resolves #FILLMEIN" (verify release bug label if applicable: https://github.com/OpenLiberty/open-liberty/wiki/Open-Liberty-Conventions).
• If this PR resolves an external Known Issue (including APARS), the description includes "Fixes #FILLMEIN" or "Resolves #FILLMEIN".

[njr-11]

!build
notification=newcomment

[LibbyBot]

Your Open Liberty Personal Build request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?pipelineId=fa92552c-58a0-4f41-ae9b-5051a118d017

Note: Target locations of links might be accessible only to IBM employees.

[LibbyBot]

Code analysis and actions

DO NOT DELETE THIS COMMENT.

• 5 FAT files were changed, added, or removed.

• Check that the build did not break the affected FAT suite(s).

• 6 product code files were changed.

• Please describe in a separate comment how you tested your changes.

• 1 messages files were changed and need an L2 review.

• @OpenLiberty/message-reviewer Please review.

  • dev/io.openliberty.data.internal.persistence/resources/io/openliberty/data/internal/persistence/resources/CWWKDMessages.nlsprops

• 1 NLS files were changed and need an ID review.

• @OpenLiberty/message-reviewer Please review.

  • dev/io.openliberty.data.internal.persistence/resources/io/openliberty/data/internal/persistence/resources/CWWKDMessages.nlsprops

[LibbyBot]

https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?pipelineId=fa92552c-58a0-4f41-ae9b-5051a118d017 - Open Liberty Personal Build completed successfully!