Braydon Fuller
589d017a14
Refactored transaction.getSerializationError to be more concise.
...
- _hasMoreOutputThanInput() and _isInvalidSatoshis() merged with getSerializationError()
- _isFeeDifferent(), _isFeeTooLarge() and _isFeeTooSmall merged with _hasFeeError()
2015-05-23 20:13:53 -04:00
David de Kloet
0b6eaf0f1e
Call getUnspentValue() only once in getSerializationError().
2015-05-23 20:13:53 -04:00
David de Kloet
3ace170ac5
Ignore fee error when unspent output is actually negative, rather than already when the check for negative unspent output is disabled.
2015-05-23 20:13:53 -04:00
David de Kloet
ac2fbe2777
When disableMoreOutputThanInput is set for getSerializationError, also disable the fee checks as the concept of a fee is meaningless when unspent output value is negative. This also allows for removing the opts from buildSkipTest again and simplifying the skip test for disableMoreOutputThanInput.
2015-05-23 20:13:53 -04:00
David de Kloet
8c5b2c851b
When checking for transaction serialization errors, check the output amount before checking fee errors. Added a test for it and also improved buildSkipTest by specifying which error to expect and using it for some tests where it wasn't used yet.
2015-05-23 20:13:53 -04:00
Braydon Fuller
b53f249d02
Merge pull request #1235 from maraoz/transaction/clearOutputs
...
add Transaction#clearOutputs
2015-05-18 16:24:02 -04:00
Braydon Fuller
c251c2bfd1
Merge pull request #1226 from dskloet/refactor/get-serialization-error
...
Simplify transaction.getSerializationError()
2015-05-18 15:08:50 -04:00
David de Kloet
99d8ac1957
Remove an unnecessary space.
2015-05-16 22:54:30 +02:00
David de Kloet
3d9560c0c5
Don't break lines at 80 characters.
2015-05-16 22:50:44 +02:00
David de Kloet
8a8412f04a
Don't break lines at 80 characters.
2015-05-16 17:06:35 +02:00
Manuel Araoz
8a13a22baf
add Transaction#clearOutputs
2015-05-15 15:29:05 -03:00
Braydon Fuller
458abe069e
Fixed issue with handling invalid output scripts
...
- Changed toObject serialization to always use a hexa string for a script
- Updated inspect method to handle a null script
- Roundtrip toObject/fromObject with an invalid script
- Additional test coverage for Output
2015-05-14 01:50:14 -04:00
Braydon Fuller
beeb294e45
Merge pull request #1217 from fanatid/fix/writeable
...
fix typo: defineProperty.writeable
2015-05-13 09:22:56 -04:00
David de Kloet
26bd5a864a
Simplify transaction.getSerializationError(). This takes care of some lint errors about number of statements and cyclomatic complexity.
2015-05-13 00:26:33 +02:00
Patrick Nagurny
de078a40d3
If output.script is InvalidBuffer, return null instead of throwing error
2015-05-12 17:23:56 -04:00
Braydon Fuller
53b6171547
Added coverage to transaction and fixed bug with serialization check.
2015-05-11 13:39:53 -04:00
Kirill Fomichev
346b2af446
delete writable for accessor property
2015-05-10 23:55:55 +03:00
David de Kloet
056f171e22
Remove the ability to disable the check that a specified fee is equal to the unspent value.
2015-05-10 01:32:28 +02:00
David de Kloet
b1e54101d3
Call the parent error of the fee errors FeeError for backwards compatibility.
2015-05-10 00:38:56 +02:00
David de Kloet
8da9c4a44a
Give the 3 fee errors a common parent error.
2015-05-10 00:32:27 +02:00
David de Kloet
d1eb190626
Introduce different kinds of FeeError to distinguish the different cases. Fix the issue uncovered by this, which is that getFee might not be the actual fee, but only an estimate, if a change address is specified but there isn't enough to pay a fee and have change.
2015-05-09 21:06:40 +02:00
David de Kloet
16dc489b08
Make sure a specified transaction fee and outputs add up to the sum of the inputs. Don't ignore the fee when it's explicitly specified.
2015-05-09 18:44:26 +02:00
Braydon Fuller
f8ea7e39cc
Included satoshis check during checked serialization.
2015-04-30 13:23:44 -04:00
Braydon Fuller
cd12164fae
Added test coverage to transaction.verify
2015-04-30 12:09:59 -04:00
Braydon Fuller
3d447ded79
Update bn.js to 2.0.4 to handle 53 bit precision number input.
2015-04-30 11:20:09 -04:00
Manuel Araoz
d556a0c33d
fix regressions
2015-04-29 18:55:42 -03:00
Manuel Araoz
dfc075e5d1
fix block parsing problem
2015-04-29 17:29:33 -03:00
Manuel Araoz
e1921429cf
remove stray console.log
2015-04-28 19:17:45 -03:00
Manuel Araoz
9a50958943
fix Block#toObject API
2015-04-28 16:42:56 -03:00
Manuel Araoz
dff0891871
add failing test for Block#toObject
2015-04-28 15:17:41 -03:00
Daniel Cousens
05a8cf9630
sighash: remove dead branch
2015-04-27 18:49:45 +10:00
Braydon Fuller
bcd4efb724
Added `checkState` for positive integer in Output satoshis
2015-04-10 18:39:08 -04:00
Braydon Fuller
3005e19cbf
Added precondition to transaction.to for a positive integer as an amount
2015-04-10 18:07:20 -04:00
Braydon Fuller
6367cbe368
Fixed typos in transaction.verify error messages
2015-04-08 12:46:18 -04:00
Kirill Fomichev
c010cb8c50
drop cached value for inputAmount and outputAmount
2015-03-31 16:22:27 +03:00
Chris Arnesen
8af63ec8d0
update transaction.getFee() JSDoc
2015-03-26 15:41:41 -07:00
eordano
294ff097a1
Add error if shuffle function doesnt return an expected result
2015-03-11 15:49:42 -03:00
eordano
a37e7b140c
Add functionality to randomize outputs
2015-03-11 15:38:53 -03:00
Manuel Araoz
b92fd915eb
increase coverage for Input and Output, fix some bugs
2015-02-27 18:46:27 -03:00
Manuel Araoz
d5fd3919c9
add Transaction#inputAmount and outputAmount
2015-02-27 16:38:48 -03:00
eordano
405f4f43df
Update dust amount
2015-02-24 17:24:22 -03:00
eordano
2f86f323c0
Add link when throwing a serialization error
2015-02-24 16:41:15 -03:00
eordano
97671abfdc
Add more docs on serialization checks
2015-02-24 16:36:08 -03:00
eordano
b81624480a
transaction: Fix verbosity of jsdocs
2015-02-24 16:34:52 -03:00
eordano
c5d7eacfac
Add check for output amount > input amount
2015-02-24 16:34:52 -03:00
eordano
f0f90c5d6a
Change unnecesarily big name
2015-02-24 13:04:40 -03:00
eordano
a6df7a175e
Better granularity on serialize() checks
2015-02-24 13:02:34 -03:00
Manuel Aráoz
e2ff93b714
Merge pull request #1083 from eordano/feature/multisigchecks
...
Transaction: Check that threshold is less than amount of signatures
2015-02-18 12:01:58 -03:00
eordano
c19d5c6501
Transaction: Check that threshold is less than amount of signatures
2015-02-18 11:52:50 -03:00
Manuel Aráoz
4c822e06c5
Merge pull request #1078 from eordano/fix/smallFee
...
Add check for small fees
2015-02-18 11:30:21 -03:00