From 69d03f060872bf9cce313f2420dc95c6d7554dec Mon Sep 17 00:00:00 2001
From: Diana Carvalho <diana@propellerheads.xyz>
Date: Fri, 14 Feb 2025 12:28:49 +0000
Subject: [PATCH] fix: Fix PLE tests that break after foundry update

--- don't change below this line ---
ENG-4222 Took 12 minutes

Took 15 seconds
---
 foundry/test/LibPrefixLengthEncodedByteArray.t.sol | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/foundry/test/LibPrefixLengthEncodedByteArray.t.sol b/foundry/test/LibPrefixLengthEncodedByteArray.t.sol
index 123f565..811dcc1 100644
--- a/foundry/test/LibPrefixLengthEncodedByteArray.t.sol
+++ b/foundry/test/LibPrefixLengthEncodedByteArray.t.sol
@@ -51,14 +51,16 @@ contract LibPrefixLengthEncodedByteArrayTest is Test {
         assertEq(this.size(multiple), 3);
     }
 
-    function testFailInvalidLength() public view {
+    function test_RevertIf_InvalidLength() public {
         // Length prefix larger than remaining data
+        vm.expectRevert();
         bytes memory invalid = hex"0004414243";
         this.next(invalid);
     }
 
-    function testFailIncompletePrefix() public view {
+    function test_RevertIf_IncompletePrefix() public {
         // Only 1 byte instead of 2 bytes prefix
+        vm.expectRevert();
         bytes memory invalid = hex"01";
         this.next(invalid);
     }