Commit 2f0471ff by Jorge Izquierdo

Make test cases that use assertJump fail if they dont fail

parent ad833fb4
...@@ -25,8 +25,9 @@ contract('BasicToken', function(accounts) { ...@@ -25,8 +25,9 @@ contract('BasicToken', function(accounts) {
try { try {
let transfer = await token.transfer(accounts[1], 101); let transfer = await token.transfer(accounts[1], 101);
} catch(error) { } catch(error) {
assertJump(error); return assertJump(error);
} }
assert.fail('should have thrown before');
}); });
}); });
...@@ -26,8 +26,9 @@ contract('LimitBalance', function(accounts) { ...@@ -26,8 +26,9 @@ contract('LimitBalance', function(accounts) {
try { try {
let limDeposit = await lb.limitedDeposit({value: amount}); let limDeposit = await lb.limitedDeposit({value: amount});
} catch(error) { } catch(error) {
assertJump(error); return assertJump(error);
} }
assert.fail('should have thrown before');
}); });
it("should allow multiple sends below limit", async function() { it("should allow multiple sends below limit", async function() {
...@@ -49,8 +50,9 @@ contract('LimitBalance', function(accounts) { ...@@ -49,8 +50,9 @@ contract('LimitBalance', function(accounts) {
try { try {
await lb.limitedDeposit({value: amount+1}) await lb.limitedDeposit({value: amount+1})
} catch(error) { } catch(error) {
assertJump(error); return assertJump(error);
} }
assert.fail('should have thrown before');
}); });
}); });
...@@ -40,8 +40,9 @@ contract('SafeMath', function(accounts) { ...@@ -40,8 +40,9 @@ contract('SafeMath', function(accounts) {
try { try {
let subtract = await safeMath.subtract(a, b); let subtract = await safeMath.subtract(a, b);
} catch(error) { } catch(error) {
assertJump(error); return assertJump(error);
} }
assert.fail('should have thrown before');
}); });
it("should throw an error on addition overflow", async function() { it("should throw an error on addition overflow", async function() {
...@@ -50,8 +51,9 @@ contract('SafeMath', function(accounts) { ...@@ -50,8 +51,9 @@ contract('SafeMath', function(accounts) {
try { try {
let add = await safeMath.add(a, b); let add = await safeMath.add(a, b);
} catch(error) { } catch(error) {
assertJump(error); return assertJump(error);
} }
assert.fail('should have thrown before');
}); });
it("should throw an error on multiplication overflow", async function() { it("should throw an error on multiplication overflow", async function() {
...@@ -60,8 +62,9 @@ contract('SafeMath', function(accounts) { ...@@ -60,8 +62,9 @@ contract('SafeMath', function(accounts) {
try { try {
let multiply = await safeMath.multiply(a, b); let multiply = await safeMath.multiply(a, b);
} catch(error) { } catch(error) {
assertJump(error); return assertJump(error);
} }
assert.fail('should have thrown before');
}); });
}); });
...@@ -32,8 +32,9 @@ contract('StandardToken', function(accounts) { ...@@ -32,8 +32,9 @@ contract('StandardToken', function(accounts) {
try { try {
let transfer = await token.transfer(accounts[1], 101); let transfer = await token.transfer(accounts[1], 101);
} catch(error) { } catch(error) {
assertJump(error); return assertJump(error);
} }
assert.fail('should have thrown before');
}); });
it("should return correct balances after transfering from another account", async function() { it("should return correct balances after transfering from another account", async function() {
...@@ -57,8 +58,9 @@ contract('StandardToken', function(accounts) { ...@@ -57,8 +58,9 @@ contract('StandardToken', function(accounts) {
try { try {
let transfer = await token.transferFrom(accounts[0], accounts[2], 100, {from: accounts[1]}); let transfer = await token.transferFrom(accounts[0], accounts[2], 100, {from: accounts[1]});
} catch (error) { } catch (error) {
assertJump(error); return assertJump(error);
} }
assert.fail('should have thrown before');
}); });
}); });
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment