Skip to content

Commit

Permalink
Finance: fix test
Browse files Browse the repository at this point in the history
With changes in PR #307 (commit bb6a18d) sending would revert instead
of failing with invalid opcode, so last test needed to change from
`assertInvalidOpcode` to `assertRevert`. We didn't realize because CI was
broken prior to PR #309.
  • Loading branch information
ßingen committed May 16, 2018
1 parent ea8425e commit 87d6ac8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion apps/finance/test/finance.js
Original file line number Diff line number Diff line change
Expand Up @@ -535,7 +535,7 @@ contract('Finance App', accounts => {
})

it('fails to deposit ETH', async() => {
return assertInvalidOpcode(async() => {
return assertRevert(async() => {
await nonInit.send(10, { gas: 3e5 })
})
})
Expand Down

0 comments on commit 87d6ac8

Please sign in to comment.