Avoid using getByTestId

[arbrandes]

Since this is a new MFE, it should reflect modern standards as much as possible. One of these is the React Testing Library query priority, which states the following:

"getByTestId: The user cannot see (or hear) these, so this is only recommended for cases where you can't match by role or text or it doesn't make sense (e.g. the text is dynamic)."

In other words, we should scour unit tests in this MFE and make sure we're not using getByTestId (or findByTestId, etc) unless absolutely necessary.

Previous PRs where I asked for this change:

• feat(authz): [FC-0099] create an custom error boundery #11 (comment)
• feat: [FC-0099] allow deleting user's roles #13 (comment)
• feat(authz): [FC-0099] Manage action feedback using the Toast component. #26 (comment)

[arbrandes]

@dcoa, @bra-i-am, would either of you want to take this on? Thanks!

[brian-smith-tcril]

Closing #17 in favor of this so we don't have 2 issues for the same thing.

PRs where I asked for this change/linked to the previous issue:

• feat: [FC-0099] add button & modal for adding new team members #3
• feat: [FC-0099] Add a new role to a library member #9