Implement PR #24 review suggestions: fix test calculations and set-prices validation

Co-authored-by: sonnyquinn24 <[email protected]>

Author: Copilot

contracts/MockERC20.sol

@@ -0,0 +1,25 @@
+// SPDX-License-Identifier: MIT
+pragma solidity ^0.8.24;
+
+import "@openzeppelin/contracts/token/ERC20/ERC20.sol";
+
+contract MockERC20 is ERC20 {
+    uint8 private _decimals;
+    
+    constructor(
+        string memory name,
+        string memory symbol,
+        uint8 decimals_
+    ) ERC20(name, symbol) {
+        _decimals = decimals_;
+        _mint(msg.sender, 1000000 * 10**decimals_); // Mint 1M tokens to deployer
+    }
+    
+    function decimals() public view virtual override returns (uint8) {
+        return _decimals;
+    }
+    
+    function mint(address to, uint256 amount) public {
+        _mint(to, amount);
+    }
+}

contracts/SEQICO.sol

@@ -14,6 +14,11 @@ contract SEQICO is Ownable {
     uint256 public pricePerTokenUSDC;
 
     event TokensPurchased(address indexed buyer, uint256 amount, string payment);
+    event PriceUpdated(string indexed paymentMethod, uint256 newPrice);
+
+    // Minimum price constants (representing $3 minimum)
+    uint256 public constant MIN_PRICE_USD_STABLECOINS = 3_000_000; // $3 with 6 decimals
+    uint256 public constant MIN_PRICE_ETH = 0.001 ether; // 0.001 ETH minimum (assuming ETH > $3000)
 
     constructor(
         address _seqToken,
@@ -26,6 +31,12 @@ contract SEQICO is Ownable {
         seqToken = IERC20(_seqToken);
         usdt = IERC20(_usdt);
         usdc = IERC20(_usdc);
+        
+        // Validate minimum prices
+        require(_pricePerTokenETH >= MIN_PRICE_ETH, "ETH price below $3 minimum");
+        require(_pricePerTokenUSDT >= MIN_PRICE_USD_STABLECOINS, "USDT price below $3 minimum");
+        require(_pricePerTokenUSDC >= MIN_PRICE_USD_STABLECOINS, "USDC price below $3 minimum");
+        
         pricePerTokenETH = _pricePerTokenETH;
         pricePerTokenUSDT = _pricePerTokenUSDT;
         pricePerTokenUSDC = _pricePerTokenUSDC;
@@ -35,6 +46,36 @@ contract SEQICO is Ownable {
         seqToken = IERC20(_seqToken);
     }
 
+    /**
+     * @dev Set the price per token for ETH purchases
+     * @param _pricePerTokenETH New price in wei per token (must be >= $3 minimum)
+     */
+    function setPriceETH(uint256 _pricePerTokenETH) external onlyOwner {
+        require(_pricePerTokenETH >= MIN_PRICE_ETH, "ETH price below $3 minimum");
+        pricePerTokenETH = _pricePerTokenETH;
+        emit PriceUpdated("ETH", _pricePerTokenETH);
+    }
+
+    /**
+     * @dev Set the price per token for USDT purchases
+     * @param _pricePerTokenUSDT New price with 6 decimals (must be >= $3)
+     */
+    function setPriceUSDT(uint256 _pricePerTokenUSDT) external onlyOwner {
+        require(_pricePerTokenUSDT >= MIN_PRICE_USD_STABLECOINS, "USDT price below $3 minimum");
+        pricePerTokenUSDT = _pricePerTokenUSDT;
+        emit PriceUpdated("USDT", _pricePerTokenUSDT);
+    }
+
+    /**
+     * @dev Set the price per token for USDC purchases
+     * @param _pricePerTokenUSDC New price with 6 decimals (must be >= $3)
+     */
+    function setPriceUSDC(uint256 _pricePerTokenUSDC) external onlyOwner {
+        require(_pricePerTokenUSDC >= MIN_PRICE_USD_STABLECOINS, "USDC price below $3 minimum");
+        pricePerTokenUSDC = _pricePerTokenUSDC;
+        emit PriceUpdated("USDC", _pricePerTokenUSDC);
+    }
+
     function buyWithETH(uint256 tokenAmount) external payable {
         require(tokenAmount > 0, "Amount must be greater than 0");
         uint256 requiredETH = pricePerTokenETH * tokenAmount;

hardhat.config.js

@@ -3,21 +3,25 @@ const config = {
   solidity: "0.8.24",
   networks: {
     hardhat: {
-      chainId: 31337
+      type: "edr-simulated"
     },
     sepolia: {
+      type: "http", 
       url: process.env.SEPOLIA_RPC_URL || `https://sepolia.infura.io/v3/${process.env.INFURA_API_KEY}`,
       accounts: process.env.PRIVATE_KEY ? [process.env.PRIVATE_KEY] : [],
     },
     mainnet: {
+      type: "http",
       url: process.env.MAINNET_RPC_URL || `https://mainnet.infura.io/v3/${process.env.INFURA_API_KEY}`,
       accounts: process.env.PRIVATE_KEY ? [process.env.PRIVATE_KEY] : [],
     },
     polygon: {
+      type: "http",
       url: process.env.POLYGON_RPC_URL || `https://polygon-mainnet.infura.io/v3/${process.env.INFURA_API_KEY}`,
       accounts: process.env.PRIVATE_KEY ? [process.env.PRIVATE_KEY] : [],
     },
     bsc: {
+      type: "http",
       url: process.env.BSC_RPC_URL || "https://bsc-dataseed1.binance.org",
       accounts: process.env.PRIVATE_KEY ? [process.env.PRIVATE_KEY] : [],
     },

package.json

@@ -4,7 +4,9 @@
   "description": "Deployment",
   "main": "index.js",
   "scripts": {
-    "test": "echo \"Error: no test specified\" && exit 1"
+    "test": "hardhat test",
+    "compile": "hardhat compile",
+    "deploy": "hardhat run scripts/deploy.js"
   },
   "keywords": [],
   "author": "",
@@ -14,5 +16,9 @@
     "@openzeppelin/contracts": "^5.4.0",
     "hardhat": "^3.0.3"
   },
-  "type": "module"
+  "type": "module",
+  "dependencies": {
+    "@nomicfoundation/hardhat-chai-matchers": "^2.1.0",
+    "solc": "^0.8.30"
+  }
 }

scripts/set-prices.js

@@ -0,0 +1,95 @@
+import { ethers } from "hardhat";
+
+async function main() {
+  // Configuration - update these addresses with your deployed contracts
+  const SEQICO_ADDRESS = "YOUR_DEPLOYED_SEQICO_ADDRESS_HERE"; // <-- Replace with your deployed SEQICO address
+  if (
+    !SEQICO_ADDRESS ||
+    SEQICO_ADDRESS === "YOUR_DEPLOYED_SEQICO_ADDRESS_HERE" ||
+    SEQICO_ADDRESS === "0x..." ||
+    !/^0x[a-fA-F0-9]{40}$/.test(SEQICO_ADDRESS)
+  ) {
+    throw new Error("❌ Please set SEQICO_ADDRESS to your deployed SEQICO contract address before running this script.");
+  }
+  
+  // New prices to set (must be >= $3 minimum)
+  const newPriceETH = ethers.parseEther("0.015"); // 0.015 ETH per token
+  const newPriceUSDT = 5_000_000; // $5 USDT (6 decimals)
+  const newPriceUSDC = 4_500_000; // $4.5 USDC (6 decimals)
+  
+  console.log("Setting new prices for SEQICO contract...");
+  console.log("Contract address:", SEQICO_ADDRESS);
+  
+  // Get the contract instance
+  const SEQICO = await ethers.getContractFactory("SEQICO");
+  const seqICO = SEQICO.attach(SEQICO_ADDRESS);
+  
+  // Verify minimum price constants
+  const minPriceETH = await seqICO.MIN_PRICE_ETH();
+  const minPriceUSD = await seqICO.MIN_PRICE_USD_STABLECOINS();
+  
+  console.log("Minimum price ETH:", ethers.formatEther(minPriceETH), "ETH");
+  console.log("Minimum price USD stablecoins:", minPriceUSD.toString(), "(representing $3)");
+  
+  // Check current prices
+  console.log("\nCurrent prices:");
+  console.log("ETH:", ethers.formatEther(await seqICO.pricePerTokenETH()), "ETH per token");
+  console.log("USDT:", (await seqICO.pricePerTokenUSDT()).toString(), "(6 decimals)");
+  console.log("USDC:", (await seqICO.pricePerTokenUSDC()).toString(), "(6 decimals)");
+  
+  // Validate new prices meet minimum requirements
+  if (newPriceETH < minPriceETH) {
+    console.error("Error: New ETH price is below minimum!");
+    return;
+  }
+  if (newPriceUSDT < minPriceUSD) {
+    console.error("Error: New USDT price is below $3 minimum!");
+    return;
+  }
+  if (newPriceUSDC < minPriceUSD) {
+    console.error("Error: New USDC price is below $3 minimum!");
+    return;
+  }
+  
+  try {
+    // Set new ETH price
+    console.log("\nSetting new ETH price...");
+    const tx1 = await seqICO.setPriceETH(newPriceETH);
+    await tx1.wait();
+    console.log("✅ ETH price updated to:", ethers.formatEther(newPriceETH), "ETH per token");
+    
+    // Set new USDT price
+    console.log("Setting new USDT price...");
+    const tx2 = await seqICO.setPriceUSDT(newPriceUSDT);
+    await tx2.wait();
+    console.log("✅ USDT price updated to:", newPriceUSDT.toString(), "($" + (newPriceUSDT / 1_000_000).toFixed(2) + ")");
+    
+    // Set new USDC price
+    console.log("Setting new USDC price...");
+    const tx3 = await seqICO.setPriceUSDC(newPriceUSDC);
+    await tx3.wait();
+    console.log("✅ USDC price updated to:", newPriceUSDC.toString(), "($" + (newPriceUSDC / 1_000_000).toFixed(2) + ")");
+    
+    console.log("\n🎉 All prices updated successfully!");
+    
+    // Verify the updates
+    console.log("\nUpdated prices:");
+    console.log("ETH:", ethers.formatEther(await seqICO.pricePerTokenETH()), "ETH per token");
+    console.log("USDT:", (await seqICO.pricePerTokenUSDT()).toString(), "(6 decimals)");
+    console.log("USDC:", (await seqICO.pricePerTokenUSDC()).toString(), "(6 decimals)");
+    
+  } catch (error) {
+    console.error("Error setting prices:", error.message);
+    if (error.message.includes("price below $3 minimum")) {
+      console.error("Make sure all prices meet the $3 minimum requirement!");
+    }
+    if (error.message.includes("OwnableUnauthorizedAccount")) {
+      console.error("Only the contract owner can set prices!");
+    }
+  }
+}
+
+main().catch((error) => {
+  console.error(error);
+  process.exitCode = 1;
+});