Test: Add token controller operator related methods unit tests

contracts/mocks/NXMTokenMock.sol

@@ -61,8 +61,9 @@ contract NXMTokenMock is INXMToken, ERC20 {
     // noop
   }
 
-  function changeOperator(address /*_newOperator*/) external returns (bool) {
-    // noop
+  function changeOperator(address _newOperator) external returns (bool) {
+    operator = _newOperator;
+    return true;
   }
 
   function lockForMemberVote(address /*_of*/, uint /*_days*/) external {

test/unit/TokenController/burnFrom.js

@@ -0,0 +1,29 @@
+const { expect } = require('chai');
+const { parseEther } = require('ethers/lib/utils');
+
+describe('burnFrom', function () {
+  it('reverts if caller is not an internal contract', async function () {
+    const { tokenController } = this.contracts;
+    const [member1] = this.accounts.members;
+
+    const amount = parseEther('10');
+    await expect(tokenController.connect(member1).burnFrom(member1.address, amount)).to.be.revertedWith(
+      'Caller is not an internal contract',
+    );
+  });
+
+  it('burns nxm from member', async function () {
+    const { tokenController, nxm } = this.contracts;
+    const [internalContract] = this.accounts.internalContracts;
+    const [member1] = this.accounts.members;
+
+    const initialBalanceMember1 = await nxm.balanceOf(member1.address);
+
+    const amount = parseEther('10');
+    await tokenController.connect(internalContract).burnFrom(member1.address, amount);
+
+    const balanceMember1 = await nxm.balanceOf(member1.address);
+
+    expect(balanceMember1).to.equal(initialBalanceMember1.sub(amount));
+  });
+});

test/unit/TokenController/changeOperator.js

@@ -0,0 +1,28 @@
+const { expect } = require('chai');
+
+describe('changeOperator', function () {
+  it('reverts if governance is not the caller', async function () {
+    const { tokenController } = this.contracts;
+    const [member] = this.accounts.members;
+
+    await expect(tokenController.connect(member).changeOperator(member.address)).to.be.revertedWith(
+      'Caller is not authorized to govern',
+    );
+  });
+
+  it('updates operator in the token contract', async function () {
+    const { tokenController, nxm } = this.contracts;
+    const [governance] = this.accounts.governanceContracts;
+    const [member] = this.accounts.members;
+
+    const initialOperator = await nxm.operator();
+
+    const newOperator = member.address;
+    await tokenController.connect(governance).changeOperator(member.address);
+
+    const operator = await nxm.operator();
+
+    expect(operator).not.equal(initialOperator);
+    expect(operator).equal(newOperator);
+  });
+});

test/unit/TokenController/index.js

@@ -20,4 +20,8 @@ describe('TokenController', function () {
   require('./burnStakingPoolNXMRewards');
   require('./burnStakedNXM');
   require('./withdrawNXMStakeAndRewards');
+  require('./changeOperator');
+  require('./operatorTransfer');
+  require('./mint');
+  require('./burnFrom');
 });

test/unit/TokenController/mint.js

@@ -0,0 +1,29 @@
+const { expect } = require('chai');
+const { parseEther } = require('ethers/lib/utils');
+
+describe('mint', function () {
+  it('reverts if caller is not an internal contract', async function () {
+    const { tokenController } = this.contracts;
+    const [member1] = this.accounts.members;
+
+    const amount = parseEther('10');
+    await expect(tokenController.connect(member1).mint(member1.address, amount)).to.be.revertedWith(
+      'Caller is not an internal contract',
+    );
+  });
+
+  it('mint nxm to member', async function () {
+    const { tokenController, nxm } = this.contracts;
+    const [internalContract] = this.accounts.internalContracts;
+    const [member1] = this.accounts.members;
+
+    const initialBalanceMember1 = await nxm.balanceOf(member1.address);
+
+    const amount = parseEther('10');
+    await tokenController.connect(internalContract).mint(member1.address, amount);
+
+    const balanceMember1 = await nxm.balanceOf(member1.address);
+
+    expect(balanceMember1).to.equal(initialBalanceMember1.add(amount));
+  });
+});

test/unit/TokenController/operatorTransfer.js

@@ -0,0 +1,50 @@
+const { expect } = require('chai');
+const { parseEther } = require('ethers/lib/utils');
+
+describe('operatorTransfer', function () {
+  it('reverts if caller is not an internal contract', async function () {
+    const { tokenController } = this.contracts;
+    const [member1, member2] = this.accounts.members;
+
+    const amount = parseEther('10');
+    await expect(
+      tokenController.connect(member1).operatorTransfer(member1.address, member2.address, amount),
+    ).to.be.revertedWith('Caller is not an internal contract');
+  });
+
+  it('transfer nxm from source address to destination address', async function () {
+    const { tokenController, nxm } = this.contracts;
+    const [internalContract] = this.accounts.internalContracts;
+    const [member1, member2] = this.accounts.members;
+
+    const initialBalanceMember1 = await nxm.balanceOf(member1.address);
+    const initialBalanceMember2 = await nxm.balanceOf(member2.address);
+
+    const amount = parseEther('10');
+    await tokenController.connect(internalContract).operatorTransfer(member1.address, member2.address, amount);
+
+    const balanceMember1 = await nxm.balanceOf(member1.address);
+    const balanceMember2 = await nxm.balanceOf(member2.address);
+
+    expect(balanceMember1).to.equal(initialBalanceMember1.sub(amount));
+    expect(balanceMember2).to.equal(initialBalanceMember2.add(amount));
+  });
+
+  it('transfer nxm from source address to token controller', async function () {
+    const { tokenController, nxm } = this.contracts;
+    const [internalContract] = this.accounts.internalContracts;
+    const [member2] = this.accounts.members;
+
+    const initialBalanceTC = await nxm.balanceOf(tokenController.address);
+    const initialBalanceMember2 = await nxm.balanceOf(member2.address);
+
+    const amount = parseEther('10');
+    await tokenController.connect(internalContract).operatorTransfer(member2.address, tokenController.address, amount);
+
+    const balanceTC = await nxm.balanceOf(tokenController.address);
+    const balanceMember2 = await nxm.balanceOf(member2.address);
+
+    expect(balanceTC).to.equal(initialBalanceTC.add(amount));
+    expect(balanceMember2).to.equal(initialBalanceMember2.sub(amount));
+  });
+});