chore: adds testing and ensures event emit matches state

contracts/contracts/core/BidderRegistry.sol

@@ -128,6 +128,8 @@ contract BidderRegistry is
     ) external nonReentrant whenNotPaused {
         uint256 currentWindow = blockTrackerContract.getCurrentWindow();
         uint256 totalAmount;
+        uint256[] memory withdrawnWindows = new uint256[](windows.length);
+        uint256[] memory withdrawnAmounts = new uint256[](windows.length);
 
         uint256 len = windows.length;
         for (uint256 i = 0; i < len; ++i) {
@@ -145,13 +147,18 @@ contract BidderRegistry is
                 usedFunds[msg.sender][uint64(blockNumber)] = 0;
             }
 
-            emit BidderWithdrawal(msg.sender, window, amount);
-
+            withdrawnWindows[i] = window;
+            withdrawnAmounts[i] = amount;
             totalAmount += amount;
         }
 
         (bool success, ) = msg.sender.call{value: totalAmount}("");
         require(success, BidderWithdrawalTransferFailed(msg.sender, totalAmount));
+
+        // Emit events only after successful transfer
+        for (uint256 i = 0; i < len; ++i) {
+            emit BidderWithdrawal(msg.sender, withdrawnWindows[i], withdrawnAmounts[i]);
+        }
     }
 
     /**

contracts/test/core/BidderRegistryTest.sol

@@ -19,23 +19,28 @@ contract BidderRegistryTest is Test {
     uint256 public feePayoutPeriodBlocks;
     BlockTracker public blockTracker;
     ProviderRegistry public providerRegistry;
+    address public oracleAccount;
 
-    /// @dev Event emitted when a bidder is registered with their staked amount
-    event BidderRegistered(address indexed bidder, uint256 indexed stakedAmount, uint256 indexed windowNumber);
-
+    event BidderRegistered(
+        address indexed bidder,
+        uint256 indexed depositedAmount,
+        uint256 indexed windowNumber
+    );
     event FeeTransfer(uint256 amount, address indexed recipient);
     event ProtocolFeeRecipientUpdated(address indexed newProtocolFeeRecipient);
     event FeePayoutPeriodBlocksUpdated(uint256 indexed newFeePayoutPeriodBlocks);
+    event BidderWithdrawal(address indexed bidder, uint256 indexed window, uint256 indexed amount);
 
     function setUp() public {
         testNumber = 42;
         feePercent = 10 * 1e16;
         minStake = 1e18 wei;
         feeRecipient = vm.addr(9);
         feePayoutPeriodBlocks = 100;
+        oracleAccount = address(this);
         address blockTrackerProxy = Upgrades.deployUUPSProxy(
             "BlockTracker.sol",
-            abi.encodeCall(BlockTracker.initialize, (address(this), address(this)))
+            abi.encodeCall(BlockTracker.initialize, (oracleAccount, address(this)))
         );
         blockTracker = BlockTracker(payable(blockTrackerProxy));
 
@@ -557,4 +562,41 @@ contract BidderRegistryTest is Test {
         vm.expectRevert(IBidderRegistry.DepositAmountIsZero.selector);
         bidderRegistry.depositForWindow{value: 0 ether}(nextWindow);   
     }
+
+    function test_WithdrawFromWindowsEventEmission() public {
+        uint256 currentWindow = blockTracker.getCurrentWindow();
+        uint256 nextWindow = currentWindow + 1;
+        uint256 depositAmount = 1000000000000000000;
+
+        // Setup: Deposit for two windows
+        vm.startPrank(bidder);
+        bidderRegistry.depositForWindow{value: depositAmount}(nextWindow);
+        bidderRegistry.depositForWindow{value: depositAmount}(nextWindow + 1);
+
+        // Move blocks forward to make windows withdrawable
+        uint256 newBlockNumber = block.number + WindowFromBlockNumber.BLOCKS_PER_WINDOW * 3;
+        vm.stopPrank();
+        vm.prank(oracleAccount);
+        blockTracker.recordL1Block(uint64(newBlockNumber), "test");
+
+        vm.startPrank(bidder);
+        // Prepare windows array for withdrawal
+        uint256[] memory windows = new uint256[](2);
+        windows[0] = nextWindow;
+        windows[1] = nextWindow + 1;
+
+        // Expect events in order
+        vm.expectEmit(true, true, true, true);
+        emit BidderWithdrawal(bidder, nextWindow, depositAmount);
+        vm.expectEmit(true, true, true, true);
+        emit BidderWithdrawal(bidder, nextWindow + 1, depositAmount);
+
+        // Perform withdrawal
+        bidderRegistry.withdrawFromWindows(windows);
+        vm.stopPrank();
+
+        // Verify state after withdrawal
+        assertEq(bidderRegistry.getDeposit(bidder, nextWindow), 0);
+        assertEq(bidderRegistry.getDeposit(bidder, nextWindow + 1), 0);
+    }
 }