Fixed matchesInGroupLeft to be consistent with documentation and matchesInGroup (#3514)

[FineAndDandy]

depends on #3513 for base test expansion exposing inconsistencies

[hlgp]

Looks good! Much more intuitive. Can we update the description here
https://github.com/NationalSecurityAgency/datawave/blob/integration/warehouse/query-core/src/main/java/datawave/query/language/functions/jexl/MatchesInGroupLeft.java#L19
to align with the changes in the new approach?

[hlgp]

One other thing to consider might be to add a -1 option to indicate "I don't know or care how many generations there are, I want them ALL to match" or make 0 = everything, 1 = first group from left, 2 = second group from left, etc.

[FineAndDandy]

Looks good! Much more intuitive. Can we update the description here https://github.com/NationalSecurityAgency/datawave/blob/integration/warehouse/query-core/src/main/java/datawave/query/language/functions/jexl/MatchesInGroupLeft.java#L19 to align with the changes in the new approach?

Yep. Good catch

[FineAndDandy]

One other thing to consider might be to add a -1 option to indicate "I don't know or care how many generations there are, I want them ALL to match" or make 0 = everything, 1 = first group from left, 2 = second group from left, etc.

so that we don't change existing behavior of 0/default I'd prefer -1 for all. I think its reasonable to add this for both #MATCHES_IN_GROUP and #MATCHES_IN_GROUP_LEFT