Extend `onlyAuthorized` to support extra functions in AccessManager

[ernestognw]

This PR adds the ability to extend the AccessManager and use the onlyAuthorized modifer for those functions as well.

PR Checklist

• [ ] Tests
• [ ] Documentation
• [ ] Changeset entry (run npx changeset add)

[changeset-bot[bot]]

🦋 Changeset detected

Latest commit: 5f81c0f3cc7489a2937ecbe0f6615b53002432b1

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
openzeppelin-solidity	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[ernestognw]

We were loosing coverage for this section:

// isTragetClosed apply to non-admin-restricted function
if (!adminRestricted && isTargetClosed(address(this))) {
    return (false, 0);
}

Turns out closing the manager is not allowed so it was unreachable. We decided to allow closing the manager so that the flag applies only to non-admin functions (i.e. functions added to the manager). This way the contract can close access to itself without getting it locked.

Should be ready