meta-txs: ignore relayer contracts for coverage measure

aragon · May 16, 2019 · c85ed73 · c85ed73
1 parent f092d3b
commit c85ed73
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/.solcover.js b/.solcover.js
@@ -2,9 +2,11 @@ const skipFiles = [
   'lib',
   'test',
   'acl/ACLSyntaxSugar.sol',
-  'common/DepositableStorage.sol', // Used in tests that send ETH
-  'common/SafeERC20.sol', // solidity-coverage fails on assembly if (https://github.com/sc-forks/solidity-coverage/issues/287)
-  'common/UnstructuredStorage.sol' // Used in tests that send ETH
+  'common/DepositableStorage.sol',  // Used in tests that send ETH
+  'common/SafeERC20.sol',           // solidity-coverage fails on assembly if (https://github.com/sc-forks/solidity-coverage/issues/287)
+  'common/UnstructuredStorage.sol', // Used in tests that send ETH
+  'relayer/Relayer.sol',            // solidity-coverage uses test-rpc which does not implement eth_signTypedData
+  'relayer/RelayedAragonApp.sol'    // solidity-coverage uses test-rpc which does not implement eth_signTypedData
 ]
 
 module.exports = {

diff --git a/contracts/relayer/Relayer.sol b/contracts/relayer/Relayer.sol
@@ -6,8 +6,6 @@ import "../lib/sig/ECDSA.sol";
 import "../lib/misc/EIP712.sol";
 import "../lib/math/SafeMath.sol";
 import "../apps/AragonApp.sol";
-import "../common/IsContract.sol";
-import "../common/TimeHelpers.sol";
 import "../common/MemoryHelpers.sol";
 import "../common/DepositableStorage.sol";
 

diff --git a/test/contracts/relayer/relayer.js b/test/contracts/relayer/relayer.js
@@ -14,7 +14,7 @@ const NOW = 1557945653
 const ONE_MONTH = 60 * 60 * 24 * 30
 const ZERO_ADDRESS = '0x0000000000000000000000000000000000000000'
 
-contract('Relayer', ([_, root, member, someone, vault, offChainRelayerService]) => {
+contract('Relayer', skipCoverage(([_, root, member, someone, vault, offChainRelayerService]) => {
   let daoFactory, dao, acl, app, relayer
   let kernelBase, aclBase, sampleAppBase, relayerBase
   let WRITING_ROLE, APP_MANAGER_ROLE, RELAYER_APP_ID
@@ -609,7 +609,7 @@ contract('Relayer', ([_, root, member, someone, vault, offChainRelayerService])
         await assertRevert(app.write(10, { from: someone }), 'APP_AUTH_FAILED')
       })
 
-      context('when the service is not allowed', () => {
+      context('when the service is allowed', () => {
         const from = offChainRelayerService
 
         beforeEach('allow service', async () => await relayer.allowService(offChainRelayerService, { from: root }))
@@ -967,4 +967,4 @@ contract('Relayer', ([_, root, member, someone, vault, offChainRelayerService])
       })
     })
   })
-})
+}))