Update test pattern

Author: Kiril Radkov

test/BondingCurve.t.sol

@@ -12,7 +12,7 @@ contract BondingCurveTest is Test {
 
     function setUp() public {}
 
-    function testPurchaseBaseCases() public pure {
+    function test_PurchaseBaseCases() public pure {
         // Test purchase with 0 ETH supply and 1 ETH input
         // Should return some tokens
         uint256 tokensOut = BondingCurve.calculatePurchaseReturn(0, 1 ether);
@@ -24,7 +24,7 @@ contract BondingCurveTest is Test {
         assert(zeroOut == 0);
     }
 
-    function testPurchasePriceIncrease() public {
+    function test_PurchasePriceIncrease() public {
         // Test that token price increases with ETH supply
         uint256 ethIn = 1 ether;
         uint256 supply1 = 10 ether;
@@ -41,13 +41,13 @@ contract BondingCurveTest is Test {
         assertTrue(priceDiff > 0 && priceDiff < 50, "Price increase should be reasonable");
     }
 
-    function testSellBaseCases() public {
+    function test_SellBaseCases() public {
         // Test sell with 0 token input
         uint256 ethOut = BondingCurve.calculateSellReturn(1 ether, 0);
         assertEq(ethOut, 0, "Should return 0 ETH for 0 tokens");
     }
 
-    function testSellPriceDecrease() public {
+    function test_SellPriceDecrease() public {
         // Setup initial state
         uint256 initialEthSupply = 50 ether;
 
@@ -60,7 +60,7 @@ contract BondingCurveTest is Test {
         assertTrue(ethOut2 < ethOut1, "Price should decrease after selling");
     }
 
-    function testCurveSymmetry() public {
+    function test_CurveSymmetry() public {
         uint256 ethSupply = 10 ether;
         uint256 ethIn = 1 ether;
 
@@ -75,7 +75,7 @@ contract BondingCurveTest is Test {
         assertTrue(ethOut > ethIn * 99 / 100, "Slippage should be reasonable");
     }
 
-    function testExtremeValues() public {
+    function test_ExtremeValues() public {
         // Test with very small amounts
         uint256 tinyEth = 1 wei;
         uint256 tokensForTiny = BondingCurve.calculatePurchaseReturn(0, tinyEth);
@@ -96,16 +96,16 @@ contract BondingCurveTest is Test {
         assertTrue(tokensForSmall < tokensForLarge, "Larger ETH input should yield more tokens");
     }
 
-    function testSellRevertOnInvalidAmount() public {
+    function test_CannotSellOnInvalidAmount() public {
         uint256 ethSupply = 10 ether;
 
         // Try to sell more tokens than possible
         uint256 maxTokens = BondingCurve.calculatePurchaseReturn(0, ethSupply);
-        vm.expectRevert();
+        vm.expectRevert(BondingCurve.FormulaInvalidTokenAmount.selector);
         BondingCurve.calculateSellReturn(ethSupply, maxTokens * 2);
     }
 
-    function testPriceProgression() public {
+    function test_PriceProgression() public {
         uint256 ethSupply = 0;
         uint256 ethIncrement = 1 ether;
         uint256 lastTokenAmount = type(uint256).max;
@@ -121,7 +121,7 @@ contract BondingCurveTest is Test {
         }
     }
 
-    function testCurveParameters() public {
+    function test_CurveParameters() public {
         // Test that the curve parameters produce expected behavior
         uint256 initialPurchase = BondingCurve.calculatePurchaseReturn(0, 1 ether);
 

test/Launchpad.t.sol

@@ -44,7 +44,7 @@ contract LaunchpadTest is Test, ArtifactStorage {
         require(address(launchpadFactory) != address(0), "LaunchpadFactory deployment failed");
     }
 
-    function testProxyCreation() public {
+    function test_ProxyCreation() public {
         string memory tokenName = "Test Token";
         string memory tokenSymbol = "TTKN";
 
@@ -85,7 +85,7 @@ contract LaunchpadTest is Test, ArtifactStorage {
         assertTrue(userEthBalance >= ethReceived, "User ETH balance mismatch after token sale");
     }
 
-    function testLiquidityMigration() public {
+    function test_LiquidityMigration() public {
         string memory tokenName = "Test Token";
         string memory tokenSymbol = "TTKN";
 

test/Proxy.t.sol

@@ -66,14 +66,14 @@ contract ProxyTest is Test, ArtifactStorage {
         vm.deal(bob, INITIAL_ETH_BALANCE);
     }
 
-    function testInitialState() public view {
+    function test_InitialState() public view {
         assertEq(proxy.ethSupply(), 0, "Initial ETH supply should be 0");
         assertEq(proxy.tokenSupply(), INITIAL_TOKEN_SUPPLY, "Initial token supply mismatch");
         assertEq(proxy.tokensLiquidity(), INITIAL_LIQUIDITY_TOKENS, "Initial liquidity tokens mismatch");
         assertFalse(proxy.isMigrated(), "Should not be migrated initially");
     }
 
-    function testSingleBuyTokens() public {
+    function test_SingleBuyTokens() public {
         uint256 buyAmount = 1 ether;
         uint256 initialTokenBalance = proxy.token().balanceOf(alice);
         uint256 initialEthSupply = proxy.ethSupply();
@@ -90,7 +90,7 @@ contract ProxyTest is Test, ArtifactStorage {
         assertEq(IERC20(address(proxy.weth())).balanceOf(address(proxy)), buyAmount, "WETH balance incorrect");
     }
 
-    function testMultipleBuysIncreasePrices() public {
+    function test_MultipleBuysIncreasePrices() public {
         uint256 buyAmount = 1 ether;
 
         // First buy
@@ -108,7 +108,7 @@ contract ProxyTest is Test, ArtifactStorage {
         assertTrue(secondBuyTokens < firstBuyTokens, "Price should increase after buys");
     }
 
-    function testSellTokens() public {
+    function test_SellTokens() public {
         // First buy tokens
         uint256 buyAmount = 1 ether;
         vm.startPrank(alice);
@@ -136,7 +136,7 @@ contract ProxyTest is Test, ArtifactStorage {
         assertEq(proxy.token().balanceOf(alice), 0, "Should have no tokens left");
     }
 
-    function testPriceDecreasesAfterSell() public {
+    function test_PriceDecreasesAfterSell() public {
         // Initial buy
         uint256 buyAmount = 2 ether;
         vm.startPrank(alice);
@@ -156,7 +156,7 @@ contract ProxyTest is Test, ArtifactStorage {
         assertTrue(priceAfterSell < priceBeforeSell, "Price should decrease after sell");
     }
 
-    function testThresholdMigration() public {
+    function test_ThresholdMigration() public {
         vm.startPrank(alice);
         uint256 thresholdAmount = proxy.THRESHOLD();
 
@@ -183,7 +183,7 @@ contract ProxyTest is Test, ArtifactStorage {
         vm.stopPrank();
     }
 
-    function testMinimumOutputAmount() public {
+    function test_MinimumOutputAmount() public {
         uint256 buyAmount = 1 ether;
         uint256 expectedTokens = proxy.getTokensOutAtCurrentSupply(buyAmount);
 

test/UniswapV2.t.sol

@@ -32,7 +32,7 @@ contract UniswapV2 is Test, ArtifactStorage {
 
     function setUp() public {}
 
-    function testWethDeployment() public {
+    function test_WethDeployment() public {
         weth = _deployBytecode(ArtifactStorage.wethBytecode);
 
         // Assert deployment succeeded
@@ -53,7 +53,7 @@ contract UniswapV2 is Test, ArtifactStorage {
         assertEq(newBalance, 0.5 ether, "Balance should be 0.5 Ether");
     }
 
-    function testFactoryDeployment() public {
+    function test_FactoryDeployment() public {
         address feeToSetter = vm.addr(1);
         bytes memory constructorArgs = abi.encode(feeToSetter);
         bytes memory bytecodeWithArgs = abi.encodePacked(ArtifactStorage.uniswapV2Factory, constructorArgs);
@@ -82,7 +82,7 @@ contract UniswapV2 is Test, ArtifactStorage {
         assertEq(pair, retrievedPair, "Retrieved pair address does not match created pair");
     }
 
-    function testRouterDeployment() public {
+    function test_RouterDeployment() public {
         // Deploy WETH
         weth = _deployBytecode(ArtifactStorage.wethBytecode);
         assertTrue(weth != address(0), "WETH deployment failed");