Removes activeStakeRequestCount from OSTComposer

contracts/gateway/OSTComposer.sol

@@ -119,9 +119,6 @@ contract OSTComposer is Organized, Mutex, ComposerInterface {
     /* Mapping of staker addresses to their StakerProxy. */
     mapping (address => StakerProxy) public stakerProxies;
 
-    /* Stores number of all active stake request per staker. */
-    mapping(address => uint256) public activeStakeRequestCount;
-
     /* Stores all the parameters of stake request based on stake request hash. */
     mapping (bytes32 => StakeRequest) public stakeRequests;
 
@@ -234,7 +231,6 @@ contract OSTComposer is Organized, Mutex, ComposerInterface {
             staker: msg.sender,
             gateway: _gateway
         });
-        activeStakeRequestCount[msg.sender] = activeStakeRequestCount[msg.sender].add(1);
 
         EIP20Interface valueToken = _gateway.valueToken();
 
@@ -286,8 +282,6 @@ contract OSTComposer is Organized, Mutex, ComposerInterface {
 
         StakerProxy stakerProxy = stakerProxies[stakeRequest.staker];
 
-        activeStakeRequestCount[stakeRequest.staker] = activeStakeRequestCount[stakeRequest.staker].sub(1);
-
         EIP20Interface valueToken = gateway.valueToken();
         require(
             valueToken.transfer(address(stakerProxy), stakeRequest.amount),
@@ -369,12 +363,6 @@ contract OSTComposer is Organized, Mutex, ComposerInterface {
         external
         onlyStakerProxy(_owner)
     {
-        // Verify if any previous stake requests are pending.
-        require(
-            activeStakeRequestCount[_owner] == 0,
-            "Stake request is active on gateways."
-        );
-
         // Resetting the proxy address of the staker.
         delete stakerProxies[_owner];
     }
@@ -396,7 +384,6 @@ contract OSTComposer is Organized, Mutex, ComposerInterface {
     {
         StakeRequest storage stakeRequest = stakeRequests[_stakeRequestHash];
         address staker = stakeRequests[_stakeRequestHash].staker;
-        activeStakeRequestCount[staker] = activeStakeRequestCount[staker].sub(1);
 
         EIP20GatewayInterface gateway = stakeRequests[_stakeRequestHash].gateway;
         uint256 amount = stakeRequests[_stakeRequestHash].amount;

contracts/test/gateway/TestOSTComposer.sol

@@ -120,22 +120,6 @@ contract TestOSTComposer is OSTComposer {
         stakerProxies[_staker] = _stakerProxy;
     }
 
-     /**
-      * @notice This is used for testing. It is used to set the
-      *         activeStakeRequestCount storage.
-      *
-      * @param _staker Address of the staker.
-      * @param _count Count of the active gateway request by the staker.
-      */
-    function setActiveStakeRequestCount(
-        address _staker,
-        uint256 _count
-    )
-        external
-    {
-        activeStakeRequestCount[_staker] = _count;
-    }
-
     /**
      * @notice This is used for testing. It is used to set the stakerProxy
      *         storage by creating the StakerProxy contract address for the staker.

package.json

@@ -52,6 +52,7 @@
     "web3": "1.0.0-beta.36"
   },
   "scripts": {
+    "update": "git submodule update --init --recursive && npm ci",
     "compile": "truffle compile",
     "compile-all": "truffle compile --all",
     "compile:ts": "tsc --target es5",

test/gateway/ost_composer/accept_stake.js

@@ -64,14 +64,9 @@ contract('OSTComposer.acceptStakeRequest() ', (accounts) => {
     activeGatewayCountForStaker = 1;
     stakerProxy = await ostComposer.generateStakerProxy.call(stakeRequest.staker);
     await ostComposer.generateStakerProxy(stakeRequest.staker);
-    await ostComposer.setActiveStakeRequestCount(stakeRequest.staker, activeGatewayCountForStaker);
   });
 
   it('should be able to successfully accept stake', async () => {
-    const activeGatewayRequestCountBeforeAS = await ostComposer.activeStakeRequestCount(
-      stakeRequest.staker,
-    );
-
     const response = await ostComposer.acceptStakeRequest(
       stakeHash,
       hashLock,
@@ -80,18 +75,6 @@ contract('OSTComposer.acceptStakeRequest() ', (accounts) => {
 
     assert.strictEqual(response.receipt.status, true, 'Receipt status is unsuccessful');
 
-    const activeGatewayRequestCountAfterAS = await ostComposer.activeStakeRequestCount(
-      stakeRequest.staker,
-    );
-
-    assert.strictEqual(
-      (activeGatewayRequestCountBeforeAS.sub(activeGatewayRequestCountAfterAS)).eqn(1),
-      true,
-      `Expected active gateway request for ${stakeRequest.staker} is `
-      + `${activeGatewayRequestCountBeforeAS.sub(activeGatewayRequestCountAfterAS)}`
-      + `but got ${activeGatewayRequestCountAfterAS}`,
-    );
-
     // Verifying the storage stakeRequestHash and stakeRequests references. After the acceptStakeRequest
     // is successful, these two storage references are deleted.
     const stakeRequestHashStorage = await ostComposer.stakeRequestHashes.call(

test/gateway/ost_composer/reject_stake_request.js

@@ -64,7 +64,6 @@ contract('OSTComposer.rejectStakeRequest() ', (accounts) => {
     gatewayCount = new BN(2);
     stakerProxy = await StakerProxy.new(stakeRequest.staker);
     await ostComposer.setStakerProxy(stakeRequest.staker, stakerProxy.address);
-    await ostComposer.setActiveStakeRequestCount(stakeRequest.staker, gatewayCount);
   });
 
   it('should be able to successfully reject stake', async () => {
@@ -75,17 +74,6 @@ contract('OSTComposer.rejectStakeRequest() ', (accounts) => {
 
     assert.strictEqual(response.receipt.status, true, 'Receipt status is unsuccessful');
 
-    const activeGatewayRequestCount = await ostComposer.activeStakeRequestCount(
-      stakeRequest.staker,
-    );
-
-    assert.strictEqual(
-      activeGatewayRequestCount.eq(gatewayCount.subn(1)),
-      true,
-      `Expected active gateway request for ${stakeRequest.staker} is ${gatewayCount.subn(1)}`
-      + `but got ${activeGatewayRequestCount}`,
-    );
-
     // Verifying the storage stakeRequestHash and stakeRequests. After the rejectStakeRequest
     // is successful, these two storage references are cleared.
     const stakeRequestHashStorage = await ostComposer.stakeRequestHashes.call(

test/gateway/ost_composer/remove_staker_proxy.js

@@ -41,7 +41,6 @@ contract('OSTComposer.removeStakerProxy() ', (accounts) => {
     expectedActiveGatewayCount = new BN(0);
     stakerProxy = accounts[10];
     await ostComposer.setStakerProxy(stakeRequest.staker, stakerProxy);
-    await ostComposer.setActiveStakeRequestCount(stakeRequest.staker, expectedActiveGatewayCount);
   });
 
   it('should be able to successfully remove staker proxy', async () => {
@@ -52,17 +51,6 @@ contract('OSTComposer.removeStakerProxy() ', (accounts) => {
 
     assert.strictEqual(response.receipt.status, true, 'Receipt status is unsuccessful');
 
-    const activeGatewayRequestCount = await ostComposer.activeStakeRequestCount(
-      stakeRequest.staker,
-    );
-
-    assert.strictEqual(
-      activeGatewayRequestCount.eq(expectedActiveGatewayCount),
-      true,
-      `Expected active gateway request for ${stakeRequest.staker} is ${expectedActiveGatewayCount}`
-      + `but got ${activeGatewayRequestCount}`,
-    );
-
     const stakerProxyAddress = await ostComposer.stakerProxies.call(stakeRequest.staker);
     assert.strictEqual(
       stakerProxyAddress,
@@ -92,17 +80,4 @@ contract('OSTComposer.removeStakerProxy() ', (accounts) => {
       'Caller is invalid proxy address.',
     );
   });
-
-  it('should fail when previous stake request is active for a staker at the gateway', async () => {
-    // It would fail for any value greater than 0.
-    await ostComposer.setActiveStakeRequestCount(stakeRequest.staker, 1);
-
-    await Utils.expectRevert(
-      ostComposer.removeStakerProxy(
-        stakeRequest.staker,
-        { from: stakerProxy },
-      ),
-      'Stake request is active on gateways.',
-    );
-  });
 });

test/gateway/ost_composer/request_stake.js

@@ -62,10 +62,6 @@ contract('OSTComposer.requestStake() ', (accounts) => {
   });
 
   it('should be able to successfully request stake', async () => {
-    const beforeRSActiveGatewayRequestCount = await ostComposer.activeStakeRequestCount(
-      stakeRequest.staker,
-    );
-
     const response = await ostComposer.requestStake(
       stakeRequest.amount,
       stakeRequest.beneficiary,
@@ -78,18 +74,6 @@ contract('OSTComposer.requestStake() ', (accounts) => {
 
     assert.strictEqual(response.receipt.status, true, 'Receipt status is unsuccessful');
 
-    const afterRSActiveGatewayRequestCount = await ostComposer.activeStakeRequestCount(
-      stakeRequest.staker,
-    );
-
-    assert.strictEqual(
-      (afterRSActiveGatewayRequestCount.sub(beforeRSActiveGatewayRequestCount)).eqn(1),
-      true,
-      `Expected active gateway request for ${stakeRequest.staker} is `
-      + `${afterRSActiveGatewayRequestCount.sub(beforeRSActiveGatewayRequestCount)} but got `
-      + `${afterRSActiveGatewayRequestCount}`,
-    );
-
     // Verifying the storage `stakeRequestHash` and `stakeRequests`.
     const stakeIntentTypeHash = getStakeRequestHash(stakeRequest, gateway);
 

test/gateway/ost_composer/revoke_stake_request.js

@@ -56,7 +56,6 @@ contract('OSTComposer.revokeStakeRequest() ', (accounts) => {
       stakeHash,
     );
     gatewayCount = new BN(2);
-    await ostComposer.setActiveStakeRequestCount(stakeRequest.staker, gatewayCount);
   });
 
   it('should be able to successfully revoke stake request', async () => {
@@ -67,17 +66,6 @@ contract('OSTComposer.revokeStakeRequest() ', (accounts) => {
 
     assert.strictEqual(response.receipt.status, true, 'Receipt status is unsuccessful');
 
-    const activeGatewayRequestCount = await ostComposer.activeStakeRequestCount(
-      stakeRequest.staker,
-    );
-
-    assert.strictEqual(
-      activeGatewayRequestCount.eq(gatewayCount.subn(1)),
-      true,
-      `Expected active gateway request for ${stakeRequest.staker} is ${gatewayCount.subn(1)}`
-      + `but got ${activeGatewayRequestCount}`,
-    );
-
     // Verifying the storage stakeRequestHash and stakeRequests. After the revokeStakeRequest
     // is successful, these two storage references are cleared.
     const stakeRequestHashStorage = await ostComposer.stakeRequestHashes.call(