Skip to content

Commit

Permalink
🎨 linting
Browse files Browse the repository at this point in the history
  • Loading branch information
Filipp Makarov authored and Filipp Makarov committed Nov 28, 2023
1 parent 9a00989 commit cde4fda
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 7 deletions.
6 changes: 4 additions & 2 deletions contracts/smart-account/base/ModuleManager.sol
Original file line number Diff line number Diff line change
Expand Up @@ -217,8 +217,10 @@ abstract contract ModuleManager is SelfAuthorized, Executor, IModuleManager {
if (module == address(0) || module == SENTINEL_MODULES) {
revert ModuleCannotBeZeroOrSentinel(module);
}
if (_modules[module] == SENTINEL_MODULES && _modules[SENTINEL_MODULES] == module)
revert CanNotDisableOnlyModule(module);
if (
_modules[module] == SENTINEL_MODULES &&
_modules[SENTINEL_MODULES] == module
) revert CanNotDisableOnlyModule(module);
if (_modules[prevModule] != module) {
revert ModuleAndPrevModuleMismatch(
module,
Expand Down
9 changes: 4 additions & 5 deletions test/smart-account/SA.Modules.specs.ts
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ describe("Modular Smart Account Modules: ", async () => {
mockToken: mockToken,
ecdsaModule: ecdsaModule,
userSA: userSA,
chainId: chainId,
chainId: chainId,
};
});

Expand Down Expand Up @@ -289,16 +289,15 @@ describe("Modular Smart Account Modules: ", async () => {
]);

const tx = await entryPoint.handleOps([userOp], alice.address);
await expect(tx).to.emit(entryPoint, "UserOperationRevertReason")
await expect(tx)
.to.emit(entryPoint, "UserOperationRevertReason")
.withArgs(
getUserOpHash(userOp, entryPoint.address, chainId),
userOp.sender,
userOp.nonce,
errorData
);
expect(await userSA.isModuleEnabled(ecdsaModule.address)).to.equal(
true
);
expect(await userSA.isModuleEnabled(ecdsaModule.address)).to.equal(true);
});
});

Expand Down

0 comments on commit cde4fda

Please sign in to comment.