Skip to content

Decouple RefundVault from RefundableCrowdsale #734

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
brunobar79 wants to merge 7 commits into from
Closed

Decouple RefundVault from RefundableCrowdsale #734

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
c47d8c9
update RefundableCrowdsale constructor
brunobar79 Feb 13, 2018
985864f
update RefundableCrowdsaleImpl constructor
brunobar79 Feb 13, 2018
d6553fb
update SampleCrowdsale
brunobar79 Feb 13, 2018
4f27546
add missing parameter to CappedCrowdsale.test
brunobar79 Feb 13, 2018
c47a793
update tests for RefundableCrowdsale
brunobar79 Feb 13, 2018
f4288a2
add {from: owner} to params
brunobar79 Feb 13, 2018
9e7a45e
keep parameters order consistent
brunobar79 Feb 14, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
keep parameters order consistent
  • Loading branch information
@brunobar79
brunobar79 committed Feb 14, 2018
commit 9e7a45e02612868aacf0872e790cff1262c83ce0
4 changes: 2 additions & 2 deletions contracts/mocks/RefundableCrowdsaleImpl.sol
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@ contract RefundableCrowdsaleImpl is RefundableCrowdsale {
uint256 _rate,
address _wallet,
uint256 _goal,
RefundVault _vault,
MintableToken _token
MintableToken _token,
RefundVault _vault
) public
Crowdsale(_startTime, _endTime, _rate, _wallet, _token)
RefundableCrowdsale(_goal, _vault)
Expand Down
4 changes: 2 additions & 2 deletions test/crowdsale/RefundableCrowdsale.test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,8 @@ contract('RefundableCrowdsale', function ([_, owner, wallet, investor]) {
rate,
wallet,
goal,
this.vault.address,
this.token.address,
this.vault.address,
{ from: owner }
);

Expand All @@ -56,8 +56,8 @@ contract('RefundableCrowdsale', function ([_, owner, wallet, investor]) {
rate,
wallet,
0,
this.vault.address,
this.token.address,
this.vault.address,
{ from: owner }
)
.should.be.rejectedWith(EVMRevert);
Expand Down