From df10bb675ceb8beb2606e2be1b4ad8ac5eb1a826 Mon Sep 17 00:00:00 2001 From: Jan-Felix Date: Thu, 3 Feb 2022 10:15:47 +0100 Subject: [PATCH] fix param order in Transfer event --- packages/contracts/contracts/TokenLock.sol | 4 ++-- packages/contracts/test/TokenLock.spec.ts | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/packages/contracts/contracts/TokenLock.sol b/packages/contracts/contracts/TokenLock.sol index 06d6ad8..5a55899 100644 --- a/packages/contracts/contracts/TokenLock.sol +++ b/packages/contracts/contracts/TokenLock.sol @@ -58,7 +58,7 @@ contract TokenLock is OwnableUpgradeable, IERC20 { revert TransferFailed(); } - emit Transfer(msg.sender, address(this), amount); + emit Transfer(address(this), msg.sender, amount); } /// @dev Withdraw tokens after the end of the locking period or during the deposit period @@ -81,7 +81,7 @@ contract TokenLock is OwnableUpgradeable, IERC20 { revert TransferFailed(); } - emit Transfer(address(this), msg.sender, amount); + emit Transfer(msg.sender, address(this), amount); } /// @dev Returns the number of decimals of the locked token diff --git a/packages/contracts/test/TokenLock.spec.ts b/packages/contracts/test/TokenLock.spec.ts index cd895de..129d80b 100644 --- a/packages/contracts/test/TokenLock.spec.ts +++ b/packages/contracts/test/TokenLock.spec.ts @@ -252,7 +252,7 @@ describe("TokenLock", () => { await expect(tokenLock.connect(user).deposit(ONE)) .to.emit(tokenLock, "Transfer") - .withArgs(user.address, tokenLock.address, ONE) + .withArgs(tokenLock.address, user.address, ONE) }) }) @@ -342,7 +342,7 @@ describe("TokenLock", () => { await expect(tokenLock.connect(user).withdraw(ONE)) .to.emit(tokenLock, "Transfer") - .withArgs(tokenLock.address, user.address, ONE) + .withArgs(user.address, tokenLock.address, ONE) }) it("allows withdrawals during the deposit period", async () => { @@ -351,7 +351,7 @@ describe("TokenLock", () => { await expect(tokenLock.connect(user).withdraw(ONE)) .to.emit(tokenLock, "Transfer") - .withArgs(tokenLock.address, user.address, ONE) + .withArgs(user.address, tokenLock.address, ONE) }) })