Move upgradeToAndCallUUPS to UUPSUpgradeable

[Amxx]

Fixes #4325 (comment)

PR Checklist

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

[changeset-bot]

🦋 Changeset detected

Latest commit: a54b74f

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	Major

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]

I just added tests to increase the coverage and pass the Codecov check. LGTM

[frangio]

We really need to agree on the guidelines for test descriptions. @Amxx and @ernestognw seem to disagree on how they should be written, based on the commit history here, and past conversations.

[ernestognw]

We really need to agree on the guidelines for test descriptions. @Amxx and @ernestognw seem to disagree on how they should be written, based on the commit history here, and past conversations.

I prefer it("does something for semantics but I'm not opposed to test("something. I just prefer the test to be read as a spec with a list of it units because it refers to the subject in a corresponding block.

Contract: MerkleProof
    verify
      ✔ returns true for a valid Merkle proof (52ms)
      ✔ returns false for an invalid Merkle proof
      ✔ returns false for a Merkle proof of invalid length

Using it('...') as part of the description makes sense to me because of this