test(KeyRegistry): add test coverage for Ownable2Step

[0xdevant]

Motivation

As mentioned on issue #262, I added few test cases to have test coverage for Ownable2Step for KeyRegistry contract.
Also formatted some contracts with forge fmt along the updates.

Change Summary

Added test cases for coverage on Ownable2Step for KeyRegistry contract, mainly are fuzz testings on transferring ownership to new owner and revert cases on unauthorized acceptance.

Merge Checklist

Choose all relevant options below by adding an x now or at any time before submitting for review

• The PR title adheres to the conventional commits standard
• The PR has been tagged with change type label(s) (i.e. documentation, feature, bugfix, or chore)
• The PR's changes adhere to all the requirements in the contribution guidelines
• All commits have been signed

---

PR-Codex overview

The focus of this PR is to refactor constructor parameter passing in multiple contracts for consistency and readability.

Detailed summary

• Refactored constructor parameter passing in Bundler, IdGateway, IdRegistry, KeyRegistry, StorageRegistry
• Adjusted array initialization in Bundler, KeyRegistryTestHelpers
• Updated event in KeyRegistry.t.sol
• Added transfer ownership functionality in KeyRegistryTest

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

[0xdevant]

Would appreciate a review from @varunsrin ✋