Merge pull request #189 from aktionariat/176-adapt-version-number-for…

…-shares

 update to correct version for shares, add version to draggable

src/shares/DraggableShares.sol

@@ -41,6 +41,12 @@ import "../ERC20/ERC20Permit2.sol";
  */
 contract DraggableShares is ERC20Draggable, ERC20Recoverable, ERC20PermitLight, ERC20Permit2 {
 
+    // Version history:
+    // 1: pre permit
+    // 2: includes permit
+    // 3: added permit2 allowance, VERSION field
+    uint8 public constant VERSION = 3;
+
     string public terms;
 
     /// Event when the terms are changed with setTerms().

src/shares/Shares.sol

@@ -54,7 +54,8 @@ contract Shares is ERC20Recoverable, ERC20Named, ERC20PermitLight, ERC20Permit2,
     // 1: everything before 2022-07-19
     // 2: added mintMany and mintManyAndCall, added VERSION field
     // 3: added permit
-    uint8 public constant VERSION = 3;
+    // 4: refactor to custom errors, added allowance for permit2
+    uint8 public constant VERSION = 4;
 
     string public terms;
 

test/test_new_token.js

@@ -122,6 +122,10 @@ describe("New Standard", () => {
       it("Should get right claim deleter", async () => {
         expect(await shares.getClaimDeleter()).to.equal(owner.address);
       });
+
+      it("Should give back newest version", async () => {
+        expect(await shares.VERSION()).to.equal(4);
+      });
     });
 
     describe("Draggable Shares", () => {
@@ -137,6 +141,10 @@ describe("New Standard", () => {
       it("Should get right claim deleter", async () => {
         expect(await draggable.getClaimDeleter()).to.equal(oracle.address);
       });
+
+      it("Should give back newest version", async () => {
+        expect(await draggable.VERSION()).to.equal(3);
+      });
     });
 
     describe("AllowlistShares", () => {