From 7c8d2d7b8938fa6d0e5c552514b03ebd0cd9e233 Mon Sep 17 00:00:00 2001
From: Roman <zajic@zajic.net>
Date: Mon, 20 Oct 2025 09:51:49 +0800
Subject: [PATCH] fix: delete MaliciousImplementation

---
 test/WakuRlnV2.t.sol | 18 ------------------
 1 file changed, 18 deletions(-)

diff --git a/test/WakuRlnV2.t.sol b/test/WakuRlnV2.t.sol
index ca81ca0..cb16dc1 100644
--- a/test/WakuRlnV2.t.sol
+++ b/test/WakuRlnV2.t.sol
@@ -68,24 +68,6 @@ contract NonUUPSContract {
 
     }
 
-// Malicious implementation for testing upgrade risks
-// This overrides _authorizeUpgrade to allow anyone (public) and adds a drain function to steal tokens
-contract MaliciousImplementation is UUPSUpgradeable, OwnableUpgradeable {
-    // Drain all balance of a token to caller (malicious)
-    function drainTokens(address token) external {
-        IERC20(token).transfer(msg.sender, IERC20(token).balanceOf(address(this)));
-    }
-
-    // Override to allow anyone to upgrade (bypassing onlyOwner)
-    function _authorizeUpgrade(address newImplementation) internal override { }
-
-    // Placeholder initializer to match layout (but malicious could ignore)
-    function initialize() public initializer {
-        __Ownable_init();
-        __UUPSUpgradeable_init();
-    }
-}
-
 contract WakuRlnV2Test is Test {
     WakuRlnV2 internal w;
     TestStableToken internal token;