Skip to content

Conversation

horsefacts
Copy link
Collaborator

@horsefacts horsefacts commented Apr 22, 2025

Add fork tests for auth keys.


PR-Codex overview

This PR introduces a new Solidity test contract, AuthKeysTest, for testing various functionalities related to authentication keys and app registration within a blockchain environment. It sets up multiple addresses, tests the registration and revocation of keys, and includes helper functions for signing metadata.

Detailed summary

  • Added // SPDX-License-Identifier: UNLICENSED and pragma solidity 0.8.21; to test/Deploy/AuthKeys.t.sol.
  • Introduced the AuthKeysTest contract extending Test.
  • Initialized multiple internal addresses and public keys.
  • Implemented setUp() function to create a blockchain fork and fund addresses.
  • Created test_e2e() function to test end-to-end key registration and revocation.
  • Added helper functions _signTransfer() and _signMetadata() for signing operations.

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

@horsefacts horsefacts force-pushed the horsefacts/auth-key-tests branch from 71ee1f0 to ccc1199 Compare May 14, 2025 14:21
@horsefacts horsefacts added the chore A task that is none of the other types label May 14, 2025
Copy link

Coverage after merging horsefacts/auth-key-tests into main will be

100.00%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   Bundler.sol100%100%100%100%
   FnameResolver.sol100%100%100%100%
   IdGateway.sol100%100%100%100%
   IdRegistry.sol100%100%100%100%
   KeyGateway.sol100%100%100%100%
   KeyRegistry.sol100%100%100%100%
   RecoveryProxy.sol100%100%100%100%
   StorageRegistry.sol100%100%100%100%
src/abstract
   EIP712.sol100%100%100%100%
   Guardians.sol100%100%100%100%
   Migration.sol96.97%80%100%100%40
   Nonces.sol100%100%100%100%
   Signatures.sol100%100%100%100%
src/validators
   SignedKeyRequestValidator.sol100%100%100%100%

@horsefacts horsefacts merged commit 918be51 into main May 14, 2025
4 checks passed
@horsefacts horsefacts deleted the horsefacts/auth-key-tests branch May 14, 2025 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore A task that is none of the other types

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant