Clarify Use of Relayer

Additionally, clarify that the transaction is executed by a relayer and not natively by the passkey signer.
safe-global · Oct 22, 2024 · b1feaa1 · b1feaa1
1 parent 8bb328c
commit b1feaa1
Showing 1 changed file with 11 additions and 6 deletions.
diff --git a/modules/passkey/test/userstories/ExecuteSafeTransaction.spec.ts b/modules/passkey/test/userstories/ExecuteSafeTransaction.spec.ts
@@ -17,13 +17,16 @@ describe('Execute Safe Transaction [@userstory]', () => {
   const setupTests = deployments.createFixture(async ({ deployments }) => {
     const { SafeProxyFactory, SafeL2, FCLP256Verifier, SafeWebAuthnSignerFactory, SafeWebAuthnSharedSigner } = await deployments.run()
 
+    const [relayer] = await ethers.getSigners()
+
     const proxyFactory = await ethers.getContractAt(SafeProxyFactory.abi, SafeProxyFactory.address)
     const singleton = await ethers.getContractAt(SafeL2.abi, SafeL2.address)
     const verifier = await ethers.getContractAt('IP256Verifier', FCLP256Verifier.address)
     const signerFactory = await ethers.getContractAt('SafeWebAuthnSignerFactory', SafeWebAuthnSignerFactory.address)
     const sharedSigner = await ethers.getContractAt('SafeWebAuthnSharedSigner', SafeWebAuthnSharedSigner.address)
 
     return {
+      relayer,
       proxyFactory,
       singleton,
       signerFactory,
@@ -35,7 +38,7 @@ describe('Execute Safe Transaction [@userstory]', () => {
   })
 
   it('should create a Safe and execute a transaction', async () => {
-    const { singleton, proxyFactory, verifier, navigator, SafeL2, signerFactory } = await setupTests()
+    const { relayer, singleton, proxyFactory, verifier, navigator, SafeL2, signerFactory } = await setupTests()
 
     // Create a WebAuthn credential to own the Safe.
     const credential = navigator.credentials.create({
@@ -127,9 +130,10 @@ describe('Execute Safe Transaction [@userstory]', () => {
     })
     const signatureData = encodeWebAuthnSignature(assertion.response)
 
-    // Execute the Safe transaction.
+    // Use relayer account to execute the signed Safe transaction.
+    const relayedSafe = safe.connect(relayer) as typeof safe
     expect(
-      await safe.execTransaction(
+      await relayedSafe.execTransaction(
         transaction.to,
         transaction.value,
         transaction.data,
@@ -151,7 +155,7 @@ describe('Execute Safe Transaction [@userstory]', () => {
   })
 
   it('should create a Safe and execute a transaction with the shared WebAuthn signer', async () => {
-    const { singleton, proxyFactory, verifier, navigator, SafeL2, sharedSigner } = await setupTests()
+    const { relayer, singleton, proxyFactory, verifier, navigator, SafeL2, sharedSigner } = await setupTests()
 
     // Create a WebAuthn credential to own the Safe.
     const credential = navigator.credentials.create({
@@ -250,9 +254,10 @@ describe('Execute Safe Transaction [@userstory]', () => {
     })
     const signatureData = encodeWebAuthnSignature(assertion.response)
 
-    // Execute the Safe transaction.
+    // Use relayer account to execute the signed Safe transaction.
+    const relayedSafe = safe.connect(relayer) as typeof safe
     expect(
-      await safe.execTransaction(
+      await relayedSafe.execTransaction(
         transaction.to,
         transaction.value,
         transaction.data,