validates contents contained in createdRBACObj in aggregated test

[onyiny-ang]

/sig multicluster

This is a follow up PR to #170 which adds validation of the contents of the RBAC Object in the crinit aggregated test

Hopefully closes issue: #114
If it doesn't, please let me know what else needs to be done to satisfy this issue and I can work on it.

[perotinus]

Thanks, @onyiny-ang! I think you might want to rebase this on top of #207 and see if it's still needed/rework it as necessary?

[perotinus]

@onyiny-ang In response to your question about testing:

• I'd avoid the RBAC-related methods for now, as they are likely to change after Remove cluster role and binding for clusters #207
• waitForAggregator is probably worth testing

[jdumars]

/ok-to-test

[perotinus]

@onyiny-ang Is this ready for review? Or should I wait to look until after #207 is in?

[onyiny-ang]

@perotinus I've rebased it to include the merged #207. All the tests still pass but it still doesn't test waitForAggregator at this point.

[perotinus]

/lgtm
/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: onyiny-ang, perotinus

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [perotinus]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment