[RHCLOUD-35963] - Added annotations for principals when filtering groups by exclude_use…

[EvanCasey13]

…rname or username

Link(s) to Jira

(https://issues.redhat.com/browse/RHCLOUD-35963)

Description of Intent of Change(s)

Added annotations for principal count when filtering group using exclude_username or username query parameters

Local Testing

How can the feature be exercised?
How can the bug be exploited and fix confirmed?
Is any special local setup required?

Checklist

• if API spec changes are required, is the spec updated?
• are there any pre/post merge actions required? if so, document here.
• are theses changes covered by unit tests?
• if warranted, are documentation changes accounted for?
• does this require migration changes?
  • if yes, are they backwards compatible?
• is there known, direct impact to dependent teams/components?
  • if yes, how will this be handled?

Secure Coding Practices Checklist Link

• https://github.com/RedHatInsights/secure-coding-checklist

Secure Coding Practices Checklist

• Input Validation
• Output Encoding
• Authentication and Password Management
• Session Management
• Access Control
• Cryptographic Practices
• Error Handling and Logging
• Data Protection
• Communication Security
• System Configuration
• Database Security
• File Management
• Memory Management
• General Coding Practices

[astrozzc]

I thought I commented on the PR but can't find it 😅
LGTM, better to have one more to take a look

[vbelchio]

@EvanCasey13
Check some combinations of tests, for the username=<user> the principalCount is always showing 1 even when there are more users inside the groups in the result.

[petracihalova]

/retest

[petracihalova]

it looks like unit tests are failing

[EvanCasey13]

/retest

[vbelchio]

QE: Passed (manual)