add refund

contracts/schemes/JoinAndQuit.sol

@@ -37,6 +37,12 @@ contract JoinAndQuit is
         uint256 indexed _refund
     );
 
+    event Refund(
+        address indexed _avatar,
+        address indexed _beneficiary,
+        uint256 indexed _refund
+    );
+
     event RedeemReputation(
         address indexed _avatar,
         bytes32 indexed _proposalId,
@@ -216,6 +222,31 @@ contract JoinAndQuit is
         emit RedeemReputation(address(avatar), _proposalId, _proposal.proposedMember, reputation);
     }
 
+    /**
+    * @dev refund refund donator if the the funding goal did not reached till the funding goal deadline.
+    * @return refund the refund amount
+    */
+    function refund() public returns(uint256 refund) {
+       // solhint-disable-next-line not-rely-on-time
+        require(now > fundingGoalDeadline, "can refund only after fundingGoalDeadline");
+        require(
+        (avatar.db(FUNDED_BEFORE_DEADLINE_KEY).hashCompareWithLengthCheck(FUNDED_BEFORE_DEADLINE_VALUE) == false),
+        "can refund only if funding goal not reached");
+        require(fundings[msg.sender].funding > 0, "no funds to refund");
+        refund = fundings[msg.sender].funding;
+        fundings[msg.sender].funding = 0;
+        if (fundingToken == IERC20(0)) {
+            require(
+            Controller(
+            avatar.owner()).sendEther(refund, msg.sender), "send ether failed");
+        } else {
+            require(
+            Controller(
+            avatar.owner()).externalTokenTransfer(fundingToken, msg.sender, refund), "send token failed");
+        }
+        emit Refund(address(avatar), msg.sender, refund);
+    }
+
     /**
     * @dev rageQuit quit from the dao.
     * can be done on any time

test/joinandquit.js

@@ -642,4 +642,34 @@ contract('JoinAndQuit', accounts => {
       }
     });
 
+    it("refund", async function() {
+      var testSetup = await setup(accounts);
+      await testSetup.standardTokenMock.approve(testSetup.joinAndQuit.address,testSetup.minFeeToJoin,{from:accounts[3]});
+      var donatorBalance = await testSetup.standardTokenMock.balanceOf(accounts[3]);
+      var tx = await testSetup.joinAndQuit.proposeToJoin(
+                                                           "description-hash",
+                                                           testSetup.minFeeToJoin,
+                                                           {from:accounts[3]});
+
+      //Vote with reputation to trigger execution
+      var proposalId = await helpers.getValueFromLogs(tx, '_proposalId',1);
+      await testSetup.joinAndQuitParams.votingMachine.absoluteVote.vote(proposalId,1,0,helpers.NULL_ADDRESS,{from:accounts[2]});
+      assert.equal(await testSetup.standardTokenMock.balanceOf(testSetup.org.avatar.address),testSetup.minFeeToJoin);
+      assert.equal((await testSetup.joinAndQuit.fundings(accounts[3])).funding,testSetup.minFeeToJoin);
+      try {
+         await testSetup.joinAndQuit.refund({from:accounts[3]});
+         assert(false, 'cannot refund before deadline');
+      } catch (ex) {
+         helpers.assertVMException(ex);
+      }
+      await helpers.increaseTime(testSetup.fundingGoalDeadline);
+      tx = await testSetup.joinAndQuit.refund({from:accounts[3]});
+      assert.equal(tx.logs.length, 1);
+      assert.equal(tx.logs[0].event, "Refund");
+      assert.equal(tx.logs[0].args._avatar, testSetup.org.avatar.address);
+      assert.equal(tx.logs[0].args._beneficiary, accounts[3]);
+      assert.equal(tx.logs[0].args._refund, testSetup.minFeeToJoin);
+      assert.equal((await testSetup.standardTokenMock.balanceOf(accounts[3])).toString(),donatorBalance.toString());
+    });
+
 });