diff --git a/packages/protocol/contracts/common/Accounts.sol b/packages/protocol/contracts/common/Accounts.sol index 12a14b32a8f..462e7a499a6 100644 --- a/packages/protocol/contracts/common/Accounts.sol +++ b/packages/protocol/contracts/common/Accounts.sol @@ -276,7 +276,7 @@ contract Accounts is */ function removeStorageRoot(uint256 index) external { require(isAccount(msg.sender), "Unknown account"); - require(index < offchainStorageRoots[msg.sender].length); + require(index < offchainStorageRoots[msg.sender].length, "Invalid storage root index"); uint256 lastIndex = offchainStorageRoots[msg.sender].length - 1; bytes memory url = offchainStorageRoots[msg.sender][index]; offchainStorageRoots[msg.sender][index] = offchainStorageRoots[msg.sender][lastIndex]; diff --git a/packages/protocol/test/common/accounts.ts b/packages/protocol/test/common/accounts.ts index a225eaa8605..f2ab25f5226 100644 --- a/packages/protocol/test/common/accounts.ts +++ b/packages/protocol/test/common/accounts.ts @@ -418,8 +418,8 @@ contract('Accounts', (accounts: string[]) => { }) describe('when there are no storage roots', async () => { - it('should revert', async () => { - await assertRevert(accountsInstance.removeStorageRoot(0)) + it('should revert with message', async () => { + await assertRevert(accountsInstance.removeStorageRoot(0), 'Invalid storage root index') }) })