fix: deposit minting

Author: jtakalai

contracts/DATAv2onPolygon.sol

@@ -26,7 +26,9 @@ contract DATAv2onPolygon is DATAv2 {
     function deposit(address user, bytes calldata depositData) external {
         require(_msgSender() == bridgeAddress, "error_onlyBridge");
         uint256 amount = abi.decode(depositData, (uint256));
-        _mint(address(this), amount);
+
+        // emits two Transfer events: 0x0 -> bridgeAddress -> user
+        _mint(bridgeAddress, amount);
         transferAndCall(user, amount, depositData);
     }
 

test/DATAv2onPolygon-test.js

@@ -0,0 +1,28 @@
+const { utils: { hexZeroPad, parseEther } } = require("ethers")
+const { expect } = require("chai")
+const { ethers: hardhatEthers } = require("hardhat")
+
+describe("DATAv2onPolygon", () => {
+    it("deposit mints tokens as expected", async () => {
+        const [bridge] = await hardhatEthers.getSigners()
+        const targetUser = "0x1234567890123456789012345678901234567890"
+        const DATAv2onPolygon = await hardhatEthers.getContractFactory("DATAv2onPolygon")
+        const token = await DATAv2onPolygon.deploy(bridge.address)
+        await token.deployed()
+
+        expect((await token.balanceOf(targetUser)).toString()).to.equal("0")
+
+        const amountBytes = hexZeroPad(parseEther("1"), 32)
+        await expect(token.connect(bridge).deposit(targetUser, amountBytes)).to.emit(token, "Transfer(address,address,uint256)")
+
+        expect((await token.balanceOf(targetUser)).toString()).to.equal(parseEther("1").toString())
+    })
+
+    it("only bridge is allowed to deposit", async () => {
+        const [bridge, another] = await hardhatEthers.getSigners()
+        const DATAv2onPolygon = await hardhatEthers.getContractFactory("DATAv2onPolygon")
+        const token = await DATAv2onPolygon.deploy(bridge.address)
+        await token.deployed()
+        await expect(token.connect(another).deposit(another.address, parseEther("1").toHexString())).to.be.revertedWith("error_onlyBridge")
+    })
+})