Merge pull request #44 from Giveth/confirmPayment
remove isProjectCanceled check from confirmPayment
This commit is contained in:
commit
84be690f03
|
@ -179,9 +179,6 @@ contract LiquidPledging is LiquidPledgingBase {
|
||||||
|
|
||||||
require(n.paymentState == PaymentState.Paying);
|
require(n.paymentState == PaymentState.Paying);
|
||||||
|
|
||||||
// Check the project is not canceled in the while.
|
|
||||||
require(!isProjectCanceled(n.owner));
|
|
||||||
|
|
||||||
uint64 idNewPledge = findOrCreatePledge(
|
uint64 idNewPledge = findOrCreatePledge(
|
||||||
n.owner,
|
n.owner,
|
||||||
n.delegationChain,
|
n.delegationChain,
|
||||||
|
|
|
@ -275,11 +275,6 @@ describe('LiquidPledging test', function () {
|
||||||
it('project2 is canceled', async () => {
|
it('project2 is canceled', async () => {
|
||||||
await liquidPledging.cancelProject(4, { from: adminProject2 });
|
await liquidPledging.cancelProject(4, { from: adminProject2 });
|
||||||
});
|
});
|
||||||
it('project2 should not be able to confirm payment', async () => {
|
|
||||||
await assertFail(async () => {
|
|
||||||
await vault.confirmPayment(1);
|
|
||||||
});
|
|
||||||
});
|
|
||||||
it('Should not be able to withdraw it', async () => {
|
it('Should not be able to withdraw it', async () => {
|
||||||
await assertFail(async () => {
|
await assertFail(async () => {
|
||||||
await liquidPledging.withdraw(12, utils.toWei(0.005), { from: giver1 });
|
await liquidPledging.withdraw(12, utils.toWei(0.005), { from: giver1 });
|
||||||
|
|
Loading…
Reference in New Issue