Feat/executefromexecutor testing

[Aboudjem]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 48.33%. Comparing base (99d3492) to head (75e4215).

Additional details and impacted files

@@                   Coverage Diff                   @@
##           feat/module-testing      #29      +/-   ##
=======================================================
+ Coverage                45.34%   48.33%   +2.98%     
=======================================================
  Files                       12       10       -2     
  Lines                      247      240       -7     
  Branches                    31       31              
=======================================================
+ Hits                       112      116       +4     
+ Misses                     135      124      -11     

Flag	Coverage Δ
foundry	48.33% <ø> (+2.98%)	⬆️
hardhat	48.33% <ø> (+2.98%)	⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

🤖 Slither Analysis Report 🔎

Slither report

Slither report

THIS CHECKLIST IS NOT COMPLETE. Use --show-ignored-findings to show all the results.
Summary
🔴 - name-reused (1 results) (High)
🟡 - unused-return (5 results) (Medium)
🔵 - shadowing-local (1 results) (Low)
🔵 - missing-zero-check (1 results) (Low)
🔵 - reentrancy-events (2 results) (Low)
ℹ️ - assembly (9 results) (Informational)
ℹ️ - dead-code (6 results) (Informational)
ℹ️ - low-level-calls (1 results) (Informational)
ℹ️ - naming-convention (1 results) (Informational)
ℹ️ - redundant-statements (1 results) (Informational)

name-reused

🔴 Impact: High
🔴 Confidence: High

• ID-0
  IModuleManager is re-used:
  • IModuleManager
  • IModuleManager

interfaces/base/IHookManager.sol#L8-L11

unused-return

🟡 Impact: Medium
🟡 Confidence: Medium

• ID-1
  ModuleManager._getExecutorsPaginated(address,uint256) ignores return value by executors.getEntriesPaginated(cursor,size)

base/ModuleManager.sol#L177-L187

• ID-2
  SmartAccount.execute(ModeCode,bytes) ignores return value by (callType,execType) = mode.decode()

SmartAccount.sol#L46-L74

• ID-3
  SmartAccount.executeFromExecutor(ModeCode,bytes) ignores return value by (callType,execType) = mode.decode()

SmartAccount.sol#L76-L129

• ID-4
  SmartAccount.supportsExecutionMode(ModeCode) ignores return value by (callType,execType) = mode.decode()

SmartAccount.sol#L192-L214

• ID-5
  ModuleManager._getValidatorsPaginated(address,uint256) ignores return value by validators.getEntriesPaginated(cursor,size)

base/ModuleManager.sol#L165-L175

shadowing-local

🔵 Impact: Low
🔴 Confidence: High

• ID-6
  SmartAccount.validateUserOp(PackedUserOperation,bytes32,uint256).nonce shadows:
  • ERC4337Account.nonce(uint192) (function)

SmartAccount.sol#L32

missing-zero-check

🔵 Impact: Low
🟡 Confidence: Medium

• ID-7
  AccountFactory.constructor(address).implementation lacks a zero-check on :
  - ACCOUNT_IMPLEMENTATION = implementation

factory/AccountFactory.sol#L11

reentrancy-events

🔵 Impact: Low
🟡 Confidence: Medium

• ID-8
  Reentrancy in SmartAccount.installModule(uint256,address,bytes):
  External calls:
  • _installValidator(module,initData)
    • IValidator(validator).onInstall(data)
  • _installExecutor(module,initData)
    • IExecutor(executor).onInstall(data)
      Event emitted after the call(s):
  • ModuleInstalled(moduleTypeId,module)

SmartAccount.sol#L140-L162

• ID-9
  Reentrancy in SmartAccount.uninstallModule(uint256,address,bytes):
  External calls:
  • _uninstallValidator(module,deInitData)
    • IValidator(validator).onUninstall(disableModuleData)
  • _uninstallExecutor(module,deInitData)
    • IExecutor(executor).onUninstall(disableModuleData)
      Event emitted after the call(s):
  • ModuleUninstalled(moduleTypeId,module)

SmartAccount.sol#L165-L180

assembly

ℹ️ Impact: Informational
🔴 Confidence: High

• ID-10
  Storage._getAccountStorage() uses assembly
  • INLINE ASM

base/Storage.sol#L18-L22

• ID-11
  SmartAccount.validateUserOp(PackedUserOperation,bytes32,uint256) uses assembly
  • INLINE ASM

SmartAccount.sol#L26-L43

• ID-12
  ModeLib.getCallType(ModeCode) uses assembly
  • INLINE ASM

lib/ModeLib.sol#L120-L124

• ID-13
  AccountExecution._tryExecuteDelegatecall(address,bytes) uses assembly
  • INLINE ASM

base/AccountExecution.sol#L108-L123

• ID-14
  ModeLib.decode(ModeCode) uses assembly
  • INLINE ASM

lib/ModeLib.sol#L87-L100

• ID-15
  AccountExecution._tryExecute(address,uint256,bytes) uses assembly
  • INLINE ASM

base/AccountExecution.sol#L71-L86

• ID-16
  AccountExecution._execute(address,uint256,bytes) uses assembly
  • INLINE ASM

base/AccountExecution.sol#L50-L69

• ID-17
  ExecLib.decodeBatch(bytes) uses assembly
  • INLINE ASM

lib/ExecLib.sol#L11-L27

• ID-18
  AccountExecution._executeDelegatecall(address,bytes) uses assembly
  • INLINE ASM

base/AccountExecution.sol#L89-L105

dead-code

ℹ️ Impact: Informational
🟡 Confidence: Medium

• ID-19
  ModuleManager._isAlreadyInitialized() is never used and should be removed

base/ModuleManager.sol#L159-L163

• ID-20
  _eqModeSelector(ModeSelector,ModeSelector) is never used and should be removed

lib/ModeLib.sol#L139-L141

• ID-21
  AccountExecution._executeDelegatecall(address,bytes) is never used and should be removed

base/AccountExecution.sol#L89-L105

• ID-22
  AccountExecution._tryExecute(Execution[]) is never used and should be removed

base/AccountExecution.sol#L38-L48

• ID-23
  AccountExecution._tryExecute(address,uint256,bytes) is never used and should be removed

base/AccountExecution.sol#L71-L86

• ID-24
  AccountExecution._tryExecuteDelegatecall(address,bytes) is never used and should be removed

base/AccountExecution.sol#L108-L123

low-level-calls

ℹ️ Impact: Informational
🔴 Confidence: High

• ID-25
  🔵 Low level call in SmartAccount.executeUserOp(PackedUserOperation,bytes32):
  • (success) = address(this).delegatecall(callData)

SmartAccount.sol#L131-L138

naming-convention

ℹ️ Impact: Informational
🔴 Confidence: High

• ID-26
  Variable AccountFactory.ACCOUNT_IMPLEMENTATION is not in mixedCase

factory/AccountFactory.sol#L9

redundant-statements

ℹ️ Impact: Informational
🔴 Confidence: High

• ID-27
  Redundant expression "additionalContext" inSmartAccount

SmartAccount.sol#L244

This comment was automatically generated by the GitHub Actions workflow.