From 6980acd59bb31d16e6cff8ba9d79e627da5e274f Mon Sep 17 00:00:00 2001
From: Mark Spanbroek <mark@spanbroek.net>
Date: Wed, 5 Feb 2025 15:14:01 +0100
Subject: [PATCH] vault: rename error

---
 contracts/vault/VaultBase.sol | 8 ++------
 test/Vault.tests.js           | 4 ++--
 2 files changed, 4 insertions(+), 8 deletions(-)

diff --git a/contracts/vault/VaultBase.sol b/contracts/vault/VaultBase.sol
index 090ce09..1c0d716 100644
--- a/contracts/vault/VaultBase.sol
+++ b/contracts/vault/VaultBase.sol
@@ -164,11 +164,7 @@ abstract contract VaultBase {
     require(lock.isLocked(), LockRequired());
 
     Account memory account = _accounts[controller][fund][recipient];
-    require(
-      account.flow.incoming == account.flow.outgoing,
-      CannotBurnFlowingTokens()
-    );
-
+    require(account.flow.incoming == account.flow.outgoing, FlowMustBeZero());
     uint128 amount = account.balance.available + account.balance.designated;
 
     lock.value -= amount;
@@ -220,5 +216,5 @@ abstract contract VaultBase {
   error ExpiryPastMaximum();
   error InvalidExpiry();
   error LockRequired();
-  error CannotBurnFlowingTokens();
+  error FlowMustBeZero();
 }
diff --git a/test/Vault.tests.js b/test/Vault.tests.js
index 104e4da..ae34a76 100644
--- a/test/Vault.tests.js
+++ b/test/Vault.tests.js
@@ -522,9 +522,9 @@ describe("Vault", function () {
       it("cannot burn tokens that are flowing", async function () {
         await vault.flow(fund, account.address, account2.address, 5)
         const burning1 = vault.burn(fund, account.address)
-        await expect(burning1).to.be.revertedWith("CannotBurnFlowingTokens")
+        await expect(burning1).to.be.revertedWith("FlowMustBeZero")
         const burning2 = vault.burn(fund, account2.address)
-        await expect(burning2).to.be.revertedWith("CannotBurnFlowingTokens")
+        await expect(burning2).to.be.revertedWith("FlowMustBeZero")
       })
 
       it("can burn tokens that are no longer flowing", async function () {