add Pausable

contracts/BNBPartyFactory.sol

@@ -38,6 +38,7 @@ contract BNBPartyFactory is BNBPartyLiquidity, ReentrancyGuard, BNBPartyManageab
         override
         nonReentrant
         insufficientBNB
+        whenNotPaused
         notZeroAddress(address(BNBPositionManager))
         returns (IERC20 newToken)
     {
@@ -54,7 +55,7 @@ contract BNBPartyFactory is BNBPartyLiquidity, ReentrancyGuard, BNBPartyManageab
 
     /// @notice Handles token swaps for the liquidity pool
     /// @param recipient The address of the entity making the exchange
-    function handleSwap(address recipient) external override onlyParty notZeroAddress(recipient) {
+    function handleSwap(address recipient) external override onlyParty notZeroAddress(recipient) whenNotPaused {
         IUniswapV3Pool pool = IUniswapV3Pool(msg.sender);
 
         uint256 WBNBBalance = WBNB.balanceOf(msg.sender);

contracts/BNBPartyManageable.sol

@@ -2,10 +2,11 @@
 pragma solidity ^0.8.0;
 
 import "./BNBPartyModifiers.sol";
+import "@openzeppelin/contracts/utils/Pausable.sol";
 
 /// @title BNBPartyManageable
 /// @notice This abstract contract provides management functions for setting position managers, swap routers, and withdrawing fees in the BNBParty system.
-abstract contract BNBPartyManageable is BNBPartyModifiers {
+abstract contract BNBPartyManageable is BNBPartyModifiers, Pausable {
     /// @notice Sets the non-fungible position managers for BNB Party and Pancakeswap V3
     /// @param _BNBPositionManager Address of the new BNB Party non-fungible position manager
     /// @param _positionManager Address of the new Pancakeswap V3 non-fungible position manager
@@ -59,6 +60,16 @@ abstract contract BNBPartyManageable is BNBPartyModifiers {
         _withdrawLPFees(liquidityPools, positionManager);
     }
 
+    /// @notice Pauses the contract
+    function pause() external onlyOwner {
+        _pause();
+    }
+
+    /// @notice Unpauses the contract
+    function unpause() external onlyOwner {
+        _unpause();
+    }
+
     /// @notice Internal function to withdraw LP fees from specified liquidity pools
     /// @param liquidityPools Array of liquidity pool addresses from which fees will be withdrawn
     /// @param manager The non-fungible position manager used to collect fees

test/BNBPartyFactory.ts

@@ -106,6 +106,39 @@ describe("BNBPartyFactory", function () {
         await bnbPartyFactory.setBNBPartySwapRouter(await BNBSwapRouter.getAddress())
     })
 
+    it("should set pause", async function () {
+        await bnbPartyFactory.pause()
+        expect(await bnbPartyFactory.paused()).to.be.true
+        await bnbPartyFactory.unpause()
+    })
+
+    it("should unpause", async function () {
+        await bnbPartyFactory.pause()
+        await bnbPartyFactory.unpause()
+        expect(await bnbPartyFactory.paused()).to.be.false
+    })
+
+    it("should revert party creation if paused", async function () {
+        await bnbPartyFactory.pause()
+        await expect(
+            bnbPartyFactory.createParty(name, symbol, { value: tokenCreationFee })
+        ).to.be.revertedWithCustomError(bnbPartyFactory, "EnforcedPause")
+        await bnbPartyFactory.unpause()
+    })
+
+    it("should revert join party if paused", async function () {
+        await bnbPartyFactory.createParty(name, symbol, { value: tokenCreationFee })
+        await bnbPartyFactory.pause()
+        const tokenId = (await BNBPositionManager.totalSupply()) - 1n
+        const position = await BNBPositionManager.positions(tokenId)
+        const MEME = position.token1 == (await weth9.getAddress()) ? position.token0 : position.token1
+        await expect(bnbPartyFactory.joinParty(MEME, 0, { value: BNBToTarget })).to.be.revertedWithCustomError(
+            bnbPartyFactory,
+            "EnforcedPause"
+        )
+        await bnbPartyFactory.unpause()
+    })
+
     describe("Second Liquidity Pool", function () {
         let MEME: string
         let tokenId: string
@@ -148,7 +181,7 @@ describe("BNBPartyFactory", function () {
             const lpAddress = await v3Factory.getPool(await weth9.getAddress(), MEME, FeeAmount.HIGH)
             const balance = await weth9.balanceOf(lpAddress)
             const percentFee = ethers.parseEther("0.14") // target 13 + 1 BNB - 1% fee
-            expect(balance).to.be.equal(BNBToTarget - returnFeeAmount - bonusFee - targetReachFee - percentFee - 1n)
+            expect(balance).to.be.equal(BNBToTarget - returnFeeAmount - bonusFee - targetReachFee - percentFee - 2n)
         })
 
         it("should send MEME to new LP", async () => {

test/WithdrawFee.ts

@@ -89,7 +89,7 @@ describe("Withdraw fees", function () {
         const lpPool = (await ethers.getContractAt("UniswapV3Pool", lpAddress)) as any as IUniswapV3Pool
         const liquidity = await lpPool.liquidity()
         const feeGrowthGlobalX128 = await lpPool.feeGrowthGlobal1X128() > 0 ? await lpPool.feeGrowthGlobal1X128() : await lpPool.feeGrowthGlobal0X128()
-        expect(await bnbPartyFactory.calculateFees(liquidity, feeGrowthGlobalX128)).to.be.equal(amountIn / 100n) // 1 % fee
+        expect(await bnbPartyFactory.calculateFees(liquidity, feeGrowthGlobalX128)).to.be.equal(amountIn / 100n - 1n) // 1 % fee
     })
 
     it("calculateFees should return fee from 5 swaps", async () => {
@@ -102,8 +102,8 @@ describe("Withdraw fees", function () {
         expect(await bnbPartyFactory.calculateFees(liquidity, feeGrowthGlobalX128)).to.be.equal(amountIn / 20n) // 1 % fee
     })
 
-    it("isToken0WBNB should return false if token0 is not WBNB", async () => {
-        expect(await bnbPartyFactory.isToken0WBNB(lpAddress)).to.be.false
+    it("isToken0WBNB should return true if token0 is WBNB", async () => {
+        expect(await bnbPartyFactory.isToken0WBNB(lpAddress)).to.be.true
     })
 
     it("isToken0WBNB should revert if set zero address", async () => {