feat(ctb): Add doOwnershipTransfer to deploy utils

[maurelian]

1. Creates a reusable function doOwnershipTransfer which deduplicates the logic for transferring ownership of the 4 system contracts to the MSD.
2. Combines the awaitCondition call for each of those transfers into a single call at the end. This provides better support for a multicall based transfer, by printing out all the info required to create a batch of transactions at once. It also still supports a 'one at a time' transfer approach, as it will check all contracts before progressing.

[changeset-bot]

⚠️ No Changeset found

Latest commit: 79706a9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

[maurelian]

Current dependencies on/for this PR:

• develop
  • PR fix(ctb): Update hardhat.json to fix hh deploy #5352
    • PR Migration script clean up #5353
      • PR feat(ctb): Add doOwnershipTransfer to deploy utils #5354 👈

This comment was auto-generated by Graphite.

[codecov]

Codecov Report

Merging #5354 (79706a9) into jm/streamline-migration/script-cleanup (153e578) will increase coverage by 0.00%.
The diff coverage is n/a.

Additional details and impacted files

@@                           Coverage Diff                           @@
##           jm/streamline-migration/script-cleanup    #5354   +/-   ##
=======================================================================
  Coverage                                   38.73%   38.73%           
=======================================================================
  Files                                         361      361           
  Lines                                       25174    25174           
  Branches                                      668      668           
=======================================================================
+ Hits                                         9750     9751    +1     
- Misses                                      14643    14644    +1     
+ Partials                                      781      779    -2     

Flag	Coverage Δ
bedrock-go-tests	35.50% <ø> (+<0.01%)	⬆️
contracts-bedrock-tests	51.20% <ø> (ø)
contracts-tests	98.86% <ø> (ø)
dtl-tests	47.15% <ø> (ø)
fault-detector-tests	33.88% <ø> (ø)
sdk-tests	38.86% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1 file with indirect coverage changes

[tynes]

This is a great addition, good work