feat: finalize contracts

Author: shottah

contracts/Token.sol

@@ -58,11 +58,11 @@ contract KolektivoTTD is Ownable, Pausable, ERC20 {
         return _isImpactPartner[account];
     }
 
-    function pause() public onlyOwner {
+    function pause() external onlyOwner {
         _pause();
     }
 
-    function unpause() public onlyOwner {
+    function unpause() external onlyOwner {
         _unpause();
     }
 

test/minting.test.ts

@@ -20,7 +20,9 @@ describe("Minting", function () {
       this.token_address = token_address;
     });
 
-    it("owner is the community steward");
+    it("owner is the community steward", async function () {
+      expect(await this.token.owner()).to.equal(this.signers.admin.address);
+    });
 
     it("should allow the community steward to mint to themselves", async function () {
       const initialSupply = await this.token.totalSupply();
@@ -40,4 +42,22 @@ describe("Minting", function () {
       await expect(this.token.connect(nonOwner).mint(nonOwner.address, 100)).to.be.reverted;
     });
   });
+
+  describe("When Paused", async function () {
+    beforeEach(async function () {
+      const { token } = await this.loadFixture(deployTokenFixture);
+      this.token = token;
+      const [_, alice] = await ethers.getSigners();
+      await this.token.connect(this.signers.admin).mint(alice.address, 100n);
+      await this.token.connect(this.signers.admin).pause();
+    });
+
+    it("cannot mint when paused", async function () {
+      const transferAmount = 100n;
+      const [_, bob] = await ethers.getSigners();
+      await expect(
+        this.token.connect(this.signers.admin).mint(bob.address, transferAmount),
+      ).to.be.revertedWithCustomError(this.token, "EnforcedPause");
+    });
+  });
 });

test/ownership.test.ts

@@ -30,6 +30,13 @@ describe("Ownership", () => {
       expect(await this.token.owner()).to.equal(newOwner.address);
     });
 
-    it("only community steward can pause transfers");
+    it("only community steward can pause", async function () {
+      const [_, alice] = await ethers.getSigners();
+      await expect(this.token.connect(this.signers.admin).pause()).to.not.be.reverted;
+      await expect(await this.token.connect(this.signers.admin).paused()).to.be.true;
+      await expect(this.token.connect(this.signers.admin).unpause()).to.not.be.reverted;
+      await expect(this.token.connect(alice).pause()).to.be.reverted;
+      await expect(this.token.connect(alice).unpause()).to.be.reverted;
+    });
   });
 });

test/transfer.test.ts

@@ -2,6 +2,7 @@ import { loadFixture } from "@nomicfoundation/hardhat-network-helpers";
 import { expect } from "chai";
 import { ethers } from "hardhat";
 
+import { KolektivoTTD, KolektivoTTD__factory } from "../types";
 import { deployTokenFixture } from "./token.fixture";
 import { Signers } from "./types";
 
@@ -33,11 +34,45 @@ describe("Transfering", function () {
     });
   });
 
-  describe("Paying @ Impact Partner", function () {
+  describe("When Paused", async function () {
+    beforeEach(async function () {
+      const { token } = await this.loadFixture(deployTokenFixture);
+      this.token = token;
+      const [_, alice] = await ethers.getSigners();
+      await this.token.connect(this.signers.admin).mint(alice.address, 100n);
+      await this.token.connect(this.signers.admin).pause();
+    });
+
+    it("can transfer when paused", async function () {
+      const transferAmount = 100n;
+      const [_, alice, bob] = await ethers.getSigners();
+      await expect(this.token.connect(alice).transfer(bob.address, transferAmount)).to.not.be.revertedWithCustomError(
+        this.token,
+        "EnforcedPause",
+      );
+    });
+  });
+
+  describe("Impact Partner", function () {
     beforeEach(async function () {
       const { token, token_address } = await this.loadFixture(deployTokenFixture);
       this.token = token;
       this.token_address = token_address;
+      const [_, partner, customer] = await ethers.getSigners();
+      await token.connect(this.signers.admin).addPartner(partner.address);
+      await token.connect(this.signers.admin).mint(customer.address, 100n);
+    });
+
+    it("impact partners can transfer", async function () {
+      const { token } = this;
+      const [_, partner, customer] = await ethers.getSigners();
+      const transferAmount = 100n;
+      await token.connect(customer).transfer(partner.address, transferAmount);
+      expect(await token.balanceOf(partner.address)).to.equal(transferAmount);
+      expect(await token.balanceOf(customer.address)).to.equal(0n);
+      await token.connect(partner).transfer(customer.address, transferAmount);
+      expect(await token.balanceOf(partner.address)).to.equal(0n);
+      expect(await token.balanceOf(customer.address)).to.equal(transferAmount);
     });
 
     it("can check if a partner is valid", async function () {
@@ -52,8 +87,6 @@ describe("Transfering", function () {
       const { token } = this;
       const [_, partner, customer] = await ethers.getSigners();
       const transferAmount = 100n;
-      await token.connect(this.signers.admin).addPartner(partner.address);
-      await token.connect(this.signers.admin).mint(customer.address, transferAmount);
       await expect(token.connect(customer).transfer(partner.address, transferAmount))
         .to.emit(token, "ImpactPartnerTransfer")
         .withArgs(customer.address, partner.address, transferAmount);