test: buyToken fail cases

Author: netpoe

hardhat/contracts/LarsKristoHellheads.sol

@@ -17,6 +17,8 @@ contract LarsKristoHellheads is ERC721Royalty {
     uint256 amount
   );
 
+  event SetTokenForSale(address indexed owner, uint256 indexed tokenId, uint256 price);
+
   address public author = 0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266; // larskristo.eth
   address public operator = 0x3C44CdDdB6a900fa2b585dd299e03d12FA4293BC; // svpervnder.eth
 
@@ -318,21 +320,13 @@ contract LarsKristoHellheads is ERC721Royalty {
    * @param price The price at which to sell the token.
    */
   function setTokenForSale(uint256 tokenId, uint256 price) public {
-    _requireTokenPriceSet(tokenId);
-
     address owner = ownerOf(tokenId);
 
     _checkAuthorized(owner, _msgSender(), tokenId);
 
     _tokenPrices[tokenId] = price;
-  }
 
-  /**
-   * @dev Returns the base URI for token metadata.
-   * @return The base URI string.
-   */
-  function _baseURI() internal view virtual override returns (string memory) {
-    return "https://blockchainassetregistry.infura-ipfs.io/ipfs/";
+    emit SetTokenForSale(owner, tokenId, price);
   }
 
   /**
@@ -353,6 +347,14 @@ contract LarsKristoHellheads is ERC721Royalty {
     return bytes(baseURI).length > 0 ? string.concat(baseURI, tokenURIhash) : "";
   }
 
+  /**
+   * @dev Returns the base URI for token metadata.
+   * @return The base URI string.
+   */
+  function _baseURI() internal view virtual override returns (string memory) {
+    return "https://blockchainassetregistry.infura-ipfs.io/ipfs/";
+  }
+
   /**
    * @dev Checks if the caller is the owner of the specified token.
    * @param tokenId The ID of the token to check ownership for.

hardhat/test/LarsKristoHellheads.ts

@@ -134,33 +134,33 @@ describe("Lease", function () {
   });
 
   it("buyToken", async function () {
-    const [author, , operator, buyer] = await ethers.getSigners();
+    const [author, someoneElse, operator, buyer] = await ethers.getSigners();
 
-    let authorBalance = await ethers.provider.getBalance(author.address);
-    let operatorBalance = await ethers.provider.getBalance(operator.address);
-    let buyerBalance = await ethers.provider.getBalance(buyer.address);
+    // let authorBalance = await ethers.provider.getBalance(author.address);
+    // let operatorBalance = await ethers.provider.getBalance(operator.address);
+    // let buyerBalance = await ethers.provider.getBalance(buyer.address);
 
-    console.log({
-      authorBalance: ethers.formatEther(authorBalance),
-      operatorBalance: ethers.formatEther(operatorBalance),
-      buyerBalance: ethers.formatEther(buyerBalance),
-    });
+    // console.log({
+    //   authorBalance: ethers.formatEther(authorBalance),
+    //   operatorBalance: ethers.formatEther(operatorBalance),
+    //   buyerBalance: ethers.formatEther(buyerBalance),
+    // });
 
     const ERC721 = await createERC721Contract();
 
     await ERC721.connect(author).setApprovalForAll(operator.address, true);
 
     await ERC721.connect(operator).setTokenForSale(0, BigInt(ethers.parseEther("1")));
 
-    authorBalance = await ethers.provider.getBalance(author.address);
-    operatorBalance = await ethers.provider.getBalance(operator.address);
-    buyerBalance = await ethers.provider.getBalance(buyer.address);
+    // authorBalance = await ethers.provider.getBalance(author.address);
+    // operatorBalance = await ethers.provider.getBalance(operator.address);
+    // buyerBalance = await ethers.provider.getBalance(buyer.address);
 
-    console.log({
-      authorBalance: ethers.formatEther(authorBalance),
-      operatorBalance: ethers.formatEther(operatorBalance),
-      buyerBalance: ethers.formatEther(buyerBalance),
-    });
+    // console.log({
+    //   authorBalance: ethers.formatEther(authorBalance),
+    //   operatorBalance: ethers.formatEther(operatorBalance),
+    //   buyerBalance: ethers.formatEther(buyerBalance),
+    // });
 
     ERC721.on("Transfer", (from, to, tokenId) => {
       // console.log({ from, to, tokenId });
@@ -187,14 +187,28 @@ describe("Lease", function () {
 
     expect(ownerOf0).to.equal(buyer.address);
 
-    authorBalance = await ethers.provider.getBalance(author.address);
-    operatorBalance = await ethers.provider.getBalance(operator.address);
-    buyerBalance = await ethers.provider.getBalance(buyer.address);
+    // authorBalance = await ethers.provider.getBalance(author.address);
+    // operatorBalance = await ethers.provider.getBalance(operator.address);
+    // buyerBalance = await ethers.provider.getBalance(buyer.address);
 
-    console.log({
-      authorBalance: ethers.formatEther(authorBalance),
-      operatorBalance: ethers.formatEther(operatorBalance),
-      buyerBalance: ethers.formatEther(buyerBalance),
-    });
+    // console.log({
+    //   authorBalance: ethers.formatEther(authorBalance),
+    //   operatorBalance: ethers.formatEther(operatorBalance),
+    //   buyerBalance: ethers.formatEther(buyerBalance),
+    // });
+
+    // only the buyer should set prices now
+    await expect(ERC721.connect(author).setTokenForSale(0, BigInt(ethers.parseEther("2")))).to.be.reverted;
+    await expect(ERC721.connect(operator).setTokenForSale(0, BigInt(ethers.parseEther("2")))).to.be.reverted;
+
+    await expect(ERC721.connect(buyer).setTokenForSale(0, BigInt(ethers.parseEther("2"))))
+      .to.emit(ERC721, "SetTokenForSale")
+      .withArgs(buyer.address, 0, BigInt(ethers.parseEther("2")));
+
+    const newPrice = await ERC721.getTokenPrice(0);
+    expect(ethers.formatEther(newPrice)).to.equal("2.0");
+
+    // too poor to buy
+    await expect(ERC721.connect(someoneElse).buyToken(0, { value: BigInt(ethers.parseEther("1")) })).to.be.reverted;
   });
 });