test: interfaces and ERC-165

.eslintrc.yml

@@ -8,6 +8,7 @@ extends:
 
 plugins:
   - '@typescript-eslint'
+  - 'unused-imports'
 
 rules:
   '@typescript-eslint/no-floating-promises':
@@ -21,6 +22,8 @@ rules:
     - argsIgnorePattern: '_'
       varsIgnorePattern: '_'
 
+  'unused-imports/no-unused-imports': 'error'
+
 ignorePatterns:
   - '*.log'
   - '*.env'

contracts/.eslintrc.yml

@@ -13,6 +13,7 @@ ignorePatterns:
   - cache
   - coverage
   - dist
+  - docs
   - typechain
   # files
   - coverage.json

contracts/package.json

@@ -9,6 +9,8 @@
   },
   "devDependencies": {
     "@aragon/osx-commons-sdk": "0.0.1-alpha.3",
+    "@aragon/osx-ethers-v1.0.0": "npm:@aragon/[email protected]",
+    "@aragon/osx-ethers-v1.3.0": "npm:@aragon/[email protected]",
     "@ethersproject/abi": "^5.7.0",
     "@ethersproject/abstract-signer": "^5.7.0",
     "@ethersproject/bignumber": "^5.7.0",

contracts/test/dao/dao.ts

@@ -0,0 +1,12 @@
+import {IDAO__factory} from '../../typechain';
+import {getInterfaceId} from '@aragon/osx-commons-sdk';
+import {IDAO__factory as IDAO_V1_0_0__factory} from '@aragon/osx-ethers-v1.0.0';
+import {expect} from 'chai';
+
+describe('IDAO', function () {
+  it('has the same interface ID as its initial version introduced in v1.0.0', async () => {
+    const current = getInterfaceId(IDAO__factory.createInterface());
+    const initial = getInterfaceId(IDAO_V1_0_0__factory.createInterface());
+    expect(current).to.equal(initial);
+  });
+});

contracts/test/helpers.ts

@@ -0,0 +1,17 @@
+import {IERC165__factory} from '../typechain';
+import {getInterfaceId} from '@aragon/osx-commons-sdk';
+import {SignerWithAddress} from '@nomiclabs/hardhat-ethers/signers';
+import {expect} from 'chai';
+import {Contract} from 'ethers';
+
+export async function erc165ComplianceTests(
+  contract: Contract,
+  signer: SignerWithAddress
+) {
+  const erc165Contract = IERC165__factory.connect(contract.address, signer);
+  const erc165InterfaceId = getInterfaceId(IERC165__factory.createInterface());
+  const emptyInterfaceId = '0xffffffff';
+
+  expect(await erc165Contract.supportsInterface(erc165InterfaceId)).to.be.true;
+  expect(await erc165Contract.supportsInterface(emptyInterfaceId)).to.be.false;
+}

contracts/test/permission/auth/auth.ts

@@ -1,7 +1,5 @@
 import {expect} from 'chai';
 
-// TODO
-
 describe.skip('auth', async () => {
   it('reverts with an error if the permission is not granted', async () => {
     expect(true).to.equal(false);

contracts/test/permission/auth/dao-authorizable.ts

@@ -1,7 +1,5 @@
 import {expect} from 'chai';
 
-// TODO
-
 describe.skip('DaoAuthorizable', async () => {
   // TODO abstract these common tests that also apply to `DaoAuthorizableUpgradeable`
   it('initializes the DAO', async () => {

contracts/test/permission/permission-condition.ts

@@ -1,21 +1,21 @@
 import {
-  IERC165__factory,
   IPermissionCondition__factory,
   IProtocolVersion__factory,
   PermissionConditionMock,
   PermissionConditionMock__factory,
 } from '../../typechain';
+import {erc165ComplianceTests} from '../helpers';
 import {getInterfaceId} from '@aragon/osx-commons-sdk';
+import {SignerWithAddress} from '@nomiclabs/hardhat-ethers/signers';
 import {expect} from 'chai';
 import {ethers} from 'hardhat';
 
-// TODO
-
 describe('PermissionCondition', async () => {
   let condition: PermissionConditionMock;
+  let deployer: SignerWithAddress;
 
   before(async () => {
-    const deployer = (await ethers.getSigners())[0];
+    [deployer] = await ethers.getSigners();
     condition = await new PermissionConditionMock__factory(deployer).deploy();
   });
 
@@ -29,14 +29,8 @@ describe('PermissionCondition', async () => {
   });
 
   describe('ERC-165', async () => {
-    it('does not support the empty interface', async () => {
-      expect(await condition.supportsInterface('0xffffffff')).to.be.false;
-    });
-
-    it('supports the `IERC165` interface', async () => {
-      const iface = IERC165__factory.createInterface();
-      expect(await condition.supportsInterface(getInterfaceId(iface))).to.be
-        .true;
+    it('supports the `ERC-165` standard', async () => {
+      await erc165ComplianceTests(condition, deployer);
     });
 
     it('supports the `IPermissionCondition` interface', async () => {