[#4] Update test descriptions

5afe · Jul 3, 2023 · b9734c1 · b9734c1
1 parent ccf7c6e
commit b9734c1
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/test/SafeProtocolMediator.spec.ts b/test/SafeProtocolMediator.spec.ts
@@ -273,7 +273,7 @@ describe("SafeProtocolMediator", async () => {
                 );
             });
 
-            it("Should process a SafeTransaction", async function () {
+            it("Should process a SafeTransaction and transfer ETH from safe to an EOA", async function () {
                 const { safeProtocolMediator, safe } = await loadFixture(deployContractsFixture);
 
                 // Enable module
@@ -301,7 +301,7 @@ describe("SafeProtocolMediator", async () => {
                 await expect(tx).to.emit(safeProtocolMediator, "ActionsExecuted").withArgs(safeAddress, safeTx.metaHash, 1);
             });
 
-            it("Should fail to process a SafeTransaction", async function () {
+            it("Should revert with ActionExecutionFailed error if Safe doesn't have enough ETH balance", async function () {
                 const { safeProtocolMediator, safe } = await loadFixture(deployContractsFixture);
 
                 // Enable module
@@ -379,7 +379,7 @@ describe("SafeProtocolMediator", async () => {
                 );
             });
 
-            it("Should revert with ModuleRequiresRootAccess", async () => {
+            it("Should revert with ModuleRequiresRootAccess if module indicates it doesn't need root access anymore", async () => {
                 const { safeProtocolMediator, safe } = await loadFixture(deployContractsFixture);
 
                 const testFallbackReceiver = await (await hre.ethers.getContractFactory("TestFallbackReceiver")).deploy(user1.address);